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

Set maximum image/icon size #158

Closed
progandy opened this issue Mar 19, 2014 · 6 comments
Closed

Set maximum image/icon size #158

progandy opened this issue Mar 19, 2014 · 6 comments

Comments

@progandy
Copy link
Contributor

Add configuration options to set an upper limit for the size of icons and images.
This is needed for icon themes (#157) and svg icons (#150) in order to choose a fitting icon size.
For embedded images (#151) there should be an additional option to allow larger images than icons.

@WishCow
Copy link

WishCow commented Feb 15, 2015

I mean no urge, just wondering is there any news on this? In some cases you can't control the images that gets sent with the notification (the most recent I encountered was with a javascript command runner, called "gulp"), and they get presented in their full size, sometimes taking up 1/3 of screen width.

Some method of scaling would be really helpful.

@Xylemon
Copy link

Xylemon commented Feb 24, 2015

I'm trying to move over from another notification daemon and this feature would be wonderful to have.

@michaelherold
Copy link

This would be a wonderful addition. Currently Guard notifications are nigh unusable due to the large size of their icons. I'm going to check with them to see why their icons are so large, but it seems like this is something that should also be handled at the notification daemon level as well.

@adnanh
Copy link

adnanh commented Aug 12, 2015

👍

@progandy
Copy link
Contributor Author

@michaelherold, as a quick fix you can modify the icons for specific notifications with a rule. Match on "icon" and set a smaller/different icon with "new_icon".

@Seeryn
Copy link
Contributor

Seeryn commented Jan 14, 2017

Fixed in ce8fe45

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants