Skip to content
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

Merged

Conversation

mikeandmore
Copy link
Contributor

Some icons are not following the XDG standard (nautilus for example). This patch uses the identity program to check the icon size.

Copy link
Member

@ThomasAdam ThomasAdam left a 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.

@mikeandmore
Copy link
Contributor Author

Yeah. The current script already uses convert, which is also from ImageMagick. We are already depending on ImageMagick (or GraphicsMagick).

Do you think it's a good idea to use PIL (Python Image Library)?

@ThomasAdam
Copy link
Member

Do you think it's a good idea to use PIL (Python Image Library)?

If you can make that work, by:

  • Removing any assumed dependence on Imagemagick;
  • Make PIL a dependency check at runtime for fvwm-menu-desktop (so that it's not a hard dependency at build-time for FVWM3`

That'd be even better!

Thanks,
Thomas

@mikeandmore
Copy link
Contributor Author

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 cairo-cffi, rsvg, and gobject-introspection just for SVG support.

@mikeandmore mikeandmore force-pushed the fvwm-menu-desktop-enhancement branch from d8140ad to 63511fa Compare June 23, 2020 16:48
@mikeandmore
Copy link
Contributor Author

OK, this patch now uses PIL or ImageMagick to scale icons. Depending on what's on the system already.

PIL is slightly faster.

Co-authored-by: Thomas Adam <ThomasAdam@users.noreply.github.com>
@ThomasAdam ThomasAdam merged commit 0e76119 into fvwmorg:master Jul 5, 2020
@mikeandmore mikeandmore deleted the fvwm-menu-desktop-enhancement branch November 29, 2020 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants