After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 727973 - W32: focused button is visibly smaller
W32: focused button is visibly smaller
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Themes
unspecified
Other All
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2014-04-10 15:39 UTC by LRN
Modified: 2014-04-10 18:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
W32: remove border on default buttons (893 bytes, patch)
2014-04-10 15:39 UTC, LRN
committed Details | Review
How dialog buttons look without the patch. (14.86 KB, image/png)
2014-04-10 15:40 UTC, LRN
  Details
How dialog buttons look with the patch. (14.67 KB, image/png)
2014-04-10 15:41 UTC, LRN
  Details

Description LRN 2014-04-10 15:39:13 UTC
This is noticeable in glade project closing confirmation dialog and in other dialogs.
The button that has keyboard focus is drawn smaller. This becomes obvious when the whole dialog loses focus (gets backdropped), which cases the button in question to be drawn without the "focused" style.
Comment 1 LRN 2014-04-10 15:39:20 UTC
Created attachment 274002 [details] [review]
W32: remove border on default buttons
Comment 2 LRN 2014-04-10 15:40:54 UTC
Created attachment 274003 [details]
How dialog buttons look without the patch.

The "Cancel" button has focus. This is not obvious, since the whole window is backdropped, and "Cancel" is drawn just like all other buttons, except that it's smaller.
Comment 3 LRN 2014-04-10 15:41:22 UTC
Created attachment 274004 [details]
How dialog buttons look with the patch.
Comment 4 LRN 2014-04-10 15:47:20 UTC
Actually, it's not that it's smaller, it's that other buttons are bigger. My understanding of this is that by default a "Can Default" button is drawn with a 1px border. When it's focused, it's drawn without that border, because default-marker is drawn outside of it, and it occupies that extra 1px.

On W32 default-marker is drawn as part of the button background by the theme part, so there's no need to reserve extra space.

By the way, this affects Raleigh (at least on my machine it does).

Also, this bug missed the 3.12 branchoff. Can the fix be backported?
Comment 5 Matthias Clasen 2014-04-10 17:35:08 UTC
Review of attachment 274002 [details] [review]:

Seems like a step in the right direction.
And yes, please put this patch on the gnome-3-12 branch too
Comment 6 LRN 2014-04-10 17:51:18 UTC
Attachment 274002 [details] pushed as 5c6e967 - W32: remove border on default buttons
Comment 7 LRN 2014-04-10 18:12:41 UTC
Attachment 274002 [details] pushed into gtk-3-12 as e41999c - W32: remove border on default buttons