-
-
Notifications
You must be signed in to change notification settings - Fork 78
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
better checks for icon size #132
better checks for icon size #132
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @mikeandmore,
Thanks. However, I do not want there to be a dependency on this script to use imagemagick
-- I think that's overkill.
I am sure there's a better way of doing this.
Yeah. The current script already uses Do you think it's a good idea to use PIL (Python Image Library)? |
If you can make that work, by:
That'd be even better! Thanks, |
Well...This is not easy. PIL doesn't support SVGs. To support SVGs, we need to introduce another SVG library that converts to PNG. The most commonly used SVG library on Linux seems to be cairo and rsvg, but that means we are depending on |
d8140ad
to
63511fa
Compare
OK, this patch now uses PIL or ImageMagick to scale icons. Depending on what's on the system already. PIL is slightly faster. |
Some icons are not following the XDG standard (nautilus for example). This patch uses the
identity
program to check the icon size.