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 583273 - add padding around status icons
add padding around status icons
Status: RESOLVED FIXED
Product: gnome-panel
Classification: Other
Component: notification area
git master
Other Linux
: Normal normal
: ---
Assigned To: Panel Maintainers
Panel Maintainers
: 140764 621341 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2009-05-19 22:38 UTC by William Jon McCann
Modified: 2011-02-22 23:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
a patch (8.25 KB, patch)
2009-10-18 03:51 UTC, Matthias Clasen
none Details | Review
corresponding patch for the notification-area-applet (8.35 KB, patch)
2009-10-18 03:52 UTC, Matthias Clasen
committed Details | Review

Description William Jon McCann 2009-05-19 22:38:22 UTC
Some status area implementations (most importantly the GNOME one) add a few pixels of padding around the socket that the status icon is added to.  This is done mainly so that the icon image itself doesn't contact the edge of the screen or the adjacent widgets.  One of the big downsides to this approach is that the active clickable area for the status icon cannot extend to the edge of the screen and therefore we lose out on one aspect of Fitt's Law.

So, one option (suggested by Owen) is to move some of this padding into the icon itself.
Comment 1 Matthias Clasen 2009-06-02 18:47:00 UTC
Looks fine to me; should probably themable in some fashion.
Comment 2 Matthias Clasen 2009-10-18 03:51:22 UTC
Created attachment 145717 [details] [review]
a patch

Here is a patch that adds padding in the GtkImage.
The amount of padding is determined by reading a _NET_SYSTEM_TRAY_PADDING
property off the manager window, in the same way that orientation and visual
are obtained.
Comment 3 Matthias Clasen 2009-10-18 03:52:48 UTC
Created attachment 145718 [details] [review]
corresponding patch for the notification-area-applet

The corresponding patch for the tray.
Comment 4 Richard Hughes 2010-05-10 14:27:04 UTC
I've rebased and merged the gtk parts into git master (2.90...)

Now, the gnome-panel parts need to be pushed upstream.
Comment 5 Dan Winship 2010-07-14 17:15:55 UTC
reassigning this to gnome-panel since there's a patch here, and not in the corresponding panel bug
Comment 6 Dan Winship 2010-07-14 17:16:24 UTC
*** Bug 140764 has been marked as a duplicate of this bug. ***
Comment 7 Dan Winship 2010-07-14 17:16:54 UTC
*** Bug 621341 has been marked as a duplicate of this bug. ***
Comment 8 Vincent Untz 2011-02-22 23:51:12 UTC
I've fixed this based on the patch, but without gconf: the padding can be set in the theme, like this:

NaTrayApplet {
    -NaTrayApplet-icon-padding: 6;
}
Comment 9 Vincent Untz 2011-02-22 23:52:09 UTC
Comment on attachment 145718 [details] [review]
corresponding patch for the notification-area-applet

Committed a modified version of this.