-
Notifications
You must be signed in to change notification settings - Fork 7
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
Change icon of result items to real program icon #23
Conversation
i was going to suggest this but forgot. I think the default kill process plugin does this, so can have a look at that plugin to see how it's done (although the kill process plugin has a lot of icons not loaded). |
A question for this is it will make windowswalker harder to be distinguished with program plugin. |
that's true actually, i never thought of it that way because i use it with a shortcut. What does PowerToys do with their icons for WindowWalker? |
they leave it like this. |
Should we maybe leave it how it is currently as well? Or make it distinguishable via subtitle e.g. 'From Window Walker- processname.exe' |
Maybe let's try implementing a small image indicating which plugin is responsible for the results? @onesounds what do you think? |
|
I am ok with current subtitle as well or my proposed one, up to @taooceros if you want change or keep it how it is now. Have this been tested and ready to go? |
close in favor of #38 |
I'm trying to get the real program icon for the results, seems work only the first query, after that fails, not sure why yet 👀