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 779570 - popover: Add more padding between checkbox and label
popover: Add more padding between checkbox and label
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Widget: GtkPopover
3.22.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
: 779385 (view as bug list)
Depends on:
Blocks: 779385
 
 
Reported: 2017-03-04 14:09 UTC by Jeremy Bicha
Modified: 2017-05-17 16:20 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
"after" screenshot (104.84 KB, image/png)
2017-05-03 18:20 UTC, Jeremy Bicha
Details

Description Jeremy Bicha 2017-03-04 14:09:43 UTC
Please add more padding between a checkbox and its label in a GtkPopover.

See this screenshot of devhelp 3.23.91 which shows the cramped "Side Panel" menu item.
https://bug779385.bugzilla-attachments.gnome.org/attachment.cgi?id=346980

This has been worked around in totem and eog but the devhelp maintainer says that he'd rather this be fixed in gtk+ because
1) every app shouldn't need to do this if they have a checkbox in their popover
2) manually setting a size request for the popover does not work very well for translations

References:
https://git.gnome.org/browse/totem/commit/?id=ed7309ea
https://git.gnome.org/browse/eog/commit/?id=a4cfef17
https://bugzilla.gnome.org/779385
Comment 1 Lapo Calamandrei 2017-05-02 15:07:51 UTC
Checks (or radios) in adwaita have a 4px margin per side, which sounds okish to me, I can probably increase the spacing, but the issue I see there is a that the check should be right aligned.
Comment 2 Lapo Calamandrei 2017-05-03 11:55:06 UTC
Looks like the check is *already* right aligned in the screenshot, hence my fault, patch coming.
Comment 3 Lapo Calamandrei 2017-05-03 15:50:24 UTC
I pushed a fix to both master and gtk-3-22. I increased the spacing from 4px to 12px and set to 0 the right side check margin, this is probably something that should be done generally, but I don't know what I may break, hence I just special cased the popover as in this particular issue.
Comment 4 Jeremy Bicha 2017-05-03 18:20:39 UTC
Created attachment 351001 [details]
"after" screenshot
Comment 5 Sébastien Wilmet 2017-05-17 16:20:25 UTC
*** Bug 779385 has been marked as a duplicate of this bug. ***