Opened 7 years ago

Closed 7 years ago

#3611 closed defect (fixed)

Pressing a button in frame "disconnects" button from palette

Reported by: Eben Owned by: marco
Priority: normal Milestone: 8.2.0 (was Update.2)
Component: sugar Version:
Keywords: Cc: benzea, mtd
Blocked By: Blocking:
Deployments affected: Action Needed:
Verified: no

Description

When pressing a button a gray rounded rect appears behind the button icon. This works fine when there is no palette open. However, when a palette is drawn, the pressed state renders a gray border around the button, which separates the button from the palette by a gray line.

This border should not be drawn in this state.

Attachments (1)

palette-transient.patch (506 bytes) - added by benzea 7 years ago.
simple patch to add the call to set_transient_for

Download all attachments as: .zip

Change History (8)

comment:1 Changed 7 years ago by benzea

  • Component changed from gtk-theme to sugar
  • Owner changed from benzea to marco

This is a sugar issue I think. Also note that the menu is the widget that is missing the gap, not the button.

comment:2 Changed 7 years ago by benzea

This is an issue with the z-order of the X windows. What is happening is that the frame window is raised above the palette one. And then the grey border of the frame is on top of the gap.

comment:3 Changed 7 years ago by Eben

  • Summary changed from Pressed state of buttons is incorrect when a palette is open to Pressing a button in frame "disconnects" button from palette

comment:4 Changed 7 years ago by jg

  • Milestone changed from Untriaged to First Deployment, V1.0

comment:5 Changed 7 years ago by benzea

  • Cc benzea added

Just found an easy way to fix this. The only thing needed need is to call gtk.Window.set_transient_for.

Changed 7 years ago by benzea

simple patch to add the call to set_transient_for

comment:6 Changed 7 years ago by mtd

  • Cc mtd added

comment:7 Changed 7 years ago by benzea

  • Resolution set to fixed
  • Status changed from new to closed

I have pushed the patch into the master branch.

Note: See TracTickets for help on using tickets.