GNOME Bugzilla – Bug 788265
notification icon too small on high resolution displays
Last modified: 2017-09-29 15:28:28 UTC
The changes made for this bug (https://bugzilla.gnome.org/show_bug.cgi?id=775763) has resulted in an icon size of 16 pixels for notification messages. On higher resolution displays the icon is very small and simply ugly. I have found that I can patch libgnome-shell.so to adjust the size for my 2K & 4K systems. It would be nice if the size could be adjusted through an extension or perhaps set via a theme.
Do you mean the icon is not scaled properly on HiDPI systems? Or do you simply disagree with the smaller size in the updated design?
I've tried scaling my fonts on a 4K display to 1.30 and a two line message is larger, but the icon is still very small. I am the author of radiotray-ng and I like to use images for my station notifications. I did briefly try HiDPI (non-fractional) scaling and it is larger so that is working as expected. To me it seems the icon should scale up to the height of the message as much as it can while maintaining its aspect ratio.
Created attachment 360627 [details] Default message icon size of 16x16
Created attachment 360628 [details] Patched calendar.js with message icon size of 48px
I've attached two examples taken from a 4K system.
Created attachment 360664 [details] [review] calendar: Make notification icons font-relative So not really a bug, but disagreement with a particular design decision - for what it's worth, your screenshot doesn't look "very small and simply ugly" to me at all. I'm not going to revert that change, but I think it makes sense for the icon to follow the text scaling. Incidentally, that change will make it possible for themes to override the icon size (if they must).
Review of attachment 360664 [details] [review]: sure ::: data/theme/gnome-shell.css @@ +922,2 @@ .message-icon-bin:rtl { + padding: 0.68em 0.68em 0.68em 0.2em; } just curious, how did you get at these numbers?
(In reply to Rui Matos from comment #7) > just curious, how did you get at these numbers? The default font size in the CSS is 11pt. At the default 96 DPI, that's 11*96/72 px (14 2/3) - and what other pixel values needs to be divided by to translate to em.
Attachment 360664 [details] pushed as 3eb80dc - calendar: Make notification icons font-relative
I should not have said "ugly". I'll attach an example of what I mean about "fuzzy". The choice of 16px is just too small for anything other than SVG images of symbols such as close/info etc.
Created attachment 360674 [details] Example of too small for non-svg images. There is plenty of room to expand the icon. Hopefully the changes made will address this.