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 762811 - Adwaita: nonsquare image buttons
Adwaita: nonsquare image buttons
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Themes
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2016-02-28 03:44 UTC by Matthias Clasen
Modified: 2016-03-03 16:13 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Adwaita: add padding to image buttons (2.58 KB, patch)
2016-03-03 14:01 UTC, Jakub Steiner
none Details | Review

Description Matthias Clasen 2016-02-28 03:44:50 UTC
Open gtk3-demo, cursors example. Observe: image buttons get top/bottom padding, but not left/right.
Comment 1 Lapo Calamandrei 2016-03-02 18:02:23 UTC
This is pretty tricky to fix, mostly since I can't rely on the image-button styleclass. Image buttons with a icon up to 24x24px inside are squared though.
Comment 2 Matthias Clasen 2016-03-02 18:30:25 UTC
why can't you rely on it ?!
Comment 3 Jakub Steiner 2016-03-03 14:00:45 UTC
As someone who doesn't quite recall all of the contexts of image-buttons and reasoning behind using min-width for image buttons, I am attaching the following patch that "makes things look reasonable to me". Lapo will likely be able to tell me why this is foolish to do. I've been wrong before...
Comment 4 Jakub Steiner 2016-03-03 14:01:06 UTC
Created attachment 322975 [details] [review]
Adwaita: add padding to image buttons

- This brings the image button metrics closer to what was in 3.18
Comment 5 Lapo Calamandrei 2016-03-03 15:10:34 UTC
Matthias I can't rely on image-button/text-button classes since those are wrongly places a lot of times.

Jakub, something like could work, If I'm not missing something, but you should check your math, 28+4+4 != 32 :-)
Comment 6 Lapo Calamandrei 2016-03-03 15:15:18 UTC
fixed with commit 08613dce92d1b0badd118c837da5013f6995e69a
Comment 7 Jakub Steiner 2016-03-03 16:13:34 UTC
Hey, maths is hard.