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

Offer white icons for dark Alfred themes #8

Closed
dixhuit opened this issue Feb 27, 2017 · 10 comments
Closed

Offer white icons for dark Alfred themes #8

dixhuit opened this issue Feb 27, 2017 · 10 comments
Assignees

Comments

@dixhuit
Copy link

dixhuit commented Feb 27, 2017

Really useful workflow here, thanks for this!

I use a dark Alfred theme and as such, the dark PNG files that you've included with this package don't look great. Would it be possible to include a set of white icons and a means to select them as a preferred option (or better yet use them automatically in dark Alfred themes)?

@dixhuit
Copy link
Author

dixhuit commented Feb 27, 2017

Or failing that, offer an option to not display icons?

@mdeboer
Copy link
Member

mdeboer commented Feb 27, 2017

@danbohea thanks! In the next upcoming version I removed the bundled icons as I never really got them to work and when they did, I found them not very appealing. I think it's too much effort to include two icon sets or to paint the icon depending on the background colour.

Instead I improved the code that finds the icons and you'll be able to specify an absolute or relative path to the icon. I'm thinking of adding the option for an environment variable (configurable through Alfred) to specify a directory with the icons from your projects (e.g. ~/Projects/Icons/.png). However, it always falls back to icon.png in your project folder(s).

I don't think it's possible to disable icons completely, but I'll need to take a look. If you have any ideas on how you want to manage your project icons, let me know. I find using a directory dedicated to project icons quite easy to work with.

@mdeboer mdeboer self-assigned this Feb 27, 2017
@dixhuit
Copy link
Author

dixhuit commented Feb 28, 2017

For those of us that don't want to manage a directory of icons for 1 Alfred extension, would it be possible to provide a further fallback whereby if "icon.png in your project folder(s)" doesn't exist then we just use the Atom app icon?

@mdeboer
Copy link
Member

mdeboer commented Feb 28, 2017

Yes definitely, I think it does that by default when no icon is found. If you think of any other method of managing icons, please let me know so I can implement it 👍

@dixhuit
Copy link
Author

dixhuit commented Feb 28, 2017

The only other thing that occurred to me was to use SVG rather than PNG, then colour could potentially be dynamically applied but I guess this all depends on Alfred and whether it even supports the use of SVG for icons.

@mdeboer
Copy link
Member

mdeboer commented Feb 28, 2017

@danbohea I checked and SVG seems to work perfectly and that would make it possible to change colour depending on the complexity of the icon. Would you like me to keep the icon set included and in SVG if I find a way to do this?

@dixhuit
Copy link
Author

dixhuit commented Feb 28, 2017

Brilliant that sounds ideal! If we could also add a preference for white or dark icon colour then that should pretty much cover all use cases.

@mdeboer
Copy link
Member

mdeboer commented Feb 28, 2017

Deal 👍

@mdeboer mdeboer added this to the v2.3.0 milestone Feb 28, 2017
@mdeboer
Copy link
Member

mdeboer commented Apr 19, 2017

Alright, I got octicon support working. It scans your Atom projects file for icons you use and converts those to PNG in the background. It will automatically recolour the icons depending on your theme and will rebuild the icons if you change it.

It'll be up for release soon, let me know what you think! 👍

@dixhuit
Copy link
Author

dixhuit commented Apr 20, 2017

Just updated. Works great! 🥇

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

No branches or pull requests

2 participants