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 683009 - Lack of padding in appmenu submenus
Lack of padding in appmenu submenus
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2012-08-30 00:29 UTC by Matthias Clasen
Modified: 2012-09-06 11:57 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
screenshot (24.50 KB, image/png)
2012-08-30 00:29 UTC, Matthias Clasen 		  Details
popupSubMenu: Only remove padding when scrollbar is shown (1.88 KB, patch)
2012-09-06 11:31 UTC, Florian Müllner 		committed Details | Review

Description Matthias Clasen 2012-08-30 00:29:36 UTC 
Created attachment 222899 [details]
screenshot

The eog appmenu has a submenu with switches in it. It looks awful.
Comment 1 Florian Müllner 2012-09-06 11:31:00 UTC 
Created attachment 223634 [details] [review]
popupSubMenu: Only remove padding when scrollbar is shown

The special padding rules for submenu items currently ensure that
content aligns properly when the scrollbar is shown. While they
work nicely for the network menu, it looks odd for non-scrolled
submenus, so make this case explicit by introducing a :scrolled
pseudo class and adjust the style rules to use it.
Comment 2 Giovanni Campagna 2012-09-06 11:36:39 UTC 
Review of attachment 223634 [details] [review]:

Ok
Comment 3 Florian Müllner 2012-09-06 11:57:45 UTC 
Attachment 223634 [details] pushed as 5847728 - popupSubMenu: Only remove padding when scrollbar is shown