-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Session icons are blurry on hidpi screens when window scaling is greater than 1 #403
Comments
Add an optional --hidpi configure flag to allow installing as a fully SVG theme, with an updated index.theme for HiDPI support. This will be seen when Scaling is set to 2x or 3x. Fixes issues where wrong icons are shown in some places with scaling enabled (like the Places sidebars in file managers), and blurry icon issues, especially with scaling enabled. The PNG non-HiDPI theme also restricted the amount of zoom steps the icons had with scaling enabled, so this should resolve that as well. Fixes shimmerproject#124 Fixes shimmerproject#317 Fixes shimmerproject#362 Fixes shimmerproject#403
Add an optional --hidpi configure flag to allow installing as a fully SVG theme, with an updated index.theme for HiDPI support. This will be seen when Scaling is set to 2x or 3x. Fixes issues where wrong icons are shown in some places with scaling enabled (like the Places sidebars in file managers), and blurry icon issues, especially with scaling enabled. The PNG non-HiDPI theme also restricted the amount of zoom steps the icons had with scaling enabled, so this should resolve that as well. Fixes shimmerproject#124 Fixes shimmerproject#317 Fixes shimmerproject#362 Fixes shimmerproject#403
Add an optional --hidpi configure flag to allow installing as a fully SVG theme, with an updated index.theme for HiDPI support. This will be seen when Scaling is set to 2x or 3x. Fixes issues where wrong icons are shown in some places with scaling enabled (like the Places sidebars in file managers), and blurry icon issues, especially with scaling enabled. The PNG non-HiDPI theme also restricted the amount of zoom steps the icons had with scaling enabled, so this should resolve that as well. Fixes shimmerproject#124 Fixes shimmerproject#317 Fixes shimmerproject#362 Fixes shimmerproject#403
Add an optional --hidpi configure flag to allow installing as a fully SVG theme, with an updated index.theme for HiDPI support. This will be seen when Scaling is set to 2x or 3x. Fixes issues where wrong icons are shown in some places with scaling enabled (like the Places sidebars in file managers), and blurry icon issues, especially with scaling enabled. The PNG non-HiDPI theme also restricted the amount of zoom steps the icons had with scaling enabled, so this should resolve that as well. Fixes shimmerproject#124 Fixes shimmerproject#317 Fixes shimmerproject#362 Fixes shimmerproject#403
Add an optional --hidpi configure flag to allow installing as a fully SVG theme, with an updated index.theme for HiDPI support. This will be seen when Scaling is set to 2x or 3x. Fixes issues where wrong icons are shown in some places with scaling enabled (like the Places sidebars in file managers), and blurry icon issues, especially with scaling enabled. The PNG non-HiDPI theme also restricted the amount of zoom steps the icons had with scaling enabled, so this should resolve that as well. Fixes shimmerproject#124 Fixes shimmerproject#317 Fixes shimmerproject#362 Fixes shimmerproject#403
@dccoder84 Sorry for the delay on this, and thanks for reporting! I added a PR for HiDPI support. If you'd like to test this branch out on your system (use --hidpi when building, Installing the HiDPI/SVG theme should scale icons better. And if the app supports scaling, it will properly use the 48px icon doubled to 96px. Xfce has been adding icon scaling to their apps during the 4.18/4.19 development. xfce4-session seems to work with this on my test (using 4.18.3), but let me know if you run into any issues. Thanks! |
Hi @newhoa, thank you but I already solved the hidpi problems of elementary xfce icon theme for all icons. Though it's only working for me personally. I did two things. First I changed all occurences of As I said most icons are displaying correct on my 4K display with gui scaling of 2x when I change the index.theme file. Among the few icons which were still blurry were the session icons and a few others. Due to the blurryness of some icons I could easily see which icons were missing in some specific resolutions for the elementary xfce icon theme. I guess I should report the few other icons which are missing in specific resolutions. |
@newhoa What do you think? Should I report the few other icons which are missing in specific resolutions as .png files. Or is this no longer necessary with your PR? |
@dccoder84 Sorry for the trouble! Glad you worked it out, but Type=Fixed is more of a band-aid/workaround for this issue and can cause widgets (buttons, etc) to shrink/expand if it can only find the fixed icon in a different size than requested. So I don't think it's something we can do by default. For proper 2x/3x scaling, Type=Scaling is required along with SVGs. Danielle Foré of elementaryOS did a nice writeup on it here if you're interested in the details. The blog explains how it works and shows how a 64px@2x icon is different than a 128px icon. A quick example, aside from the border/alignment stuff, is that in the Thunar sidebar, we have the circle-with-down-arrow for Downloads at 16x and 24x, and the folder design in higher sizes. If you use 2x scaling with a non-HiDPI theme, you will get 32px and 48px folder icons. But with the Hi-DPI theme (and Thunar 4.19 with HiDPI icon support - anything after this commit I believe), you should get the 16px and 24px circle-with-down-arrow doubled. And no need to report PNG icons that are missing when using scaling (but thanks for offering, and for the other reports). With this PR, the SVG/Hi-DPI is optional, so by default it is still PNG. But the PNG theme shouldn't be used for scaling anymore once we get a proper SVG option. If you get time, maybe give that branch and the |
@newhoa Sorry but how do I test your branch? Where can I apply the |
@newhoa Please forget my last comment. I downloaded the wrong data. I now found there is an option for With SVG icons, your new branch with --hidpi enabled: My personal index.theme hack based on PNG files: Open the images with a program which displays them in their native size. Firefox is bad at this because Firefox ignores GUI scaling and so with GUI scaling set to for example 2x will show images in Firefox with 200% scaling if you only open the image (no web page) in Firefox. A recent version of Ristretto does a much better job ;) So with Ristretto you can see a clear difference between the images. |
Maybe that rendering of the SVG icon is intended that way. But if so I don't like it. Actually the icons don't look blurry but they look like they are drawn with a thicker pencil. |
@dccoder84 Thank you for testing and for the screenshots! Some questions if you have time:
Thanks! |
You're welcome!
|
Install SVG-based HiDPI theme alongside the PNG-based theme. This should be used for better icon scaling, specifically when Scaling is set to 2x or 3x. PNG may be a bit faster to load, so the SVG HiDPI theme is optional. Using the HiDPI theme will fix issues where the wrong icons are shown in some places with scaling enabled (like the Places sidebars in file managers). It should also fix and blurry icon issues, especially with scaling enabled. The PNG non-HiDPI theme restricted the amount of zoom steps the icons had in file managers with scaling enabled, so this should resolve that as well. Fixes shimmerproject#124 Fixes shimmerproject#317 Fixes shimmerproject#362 Fixes shimmerproject#403
From what I see the icons are available with a maximum size of 48 px but for example I would need 96 px when window scaling is set to 2x.
For example the icon system-shutdown is only available in these sizes:
So can we have higher res PNGs of these icons with 96 px?
The text was updated successfully, but these errors were encountered: