Skip to content

Conversation

@taooceros
Copy link
Member

@taooceros taooceros commented Feb 12, 2021

It will help scaling, although seems that our icon is too complex to display well in 16*16 scale.

#86

@taooceros taooceros added the kind/ui related to UI, icons, themes, etc label Feb 12, 2021
@jjw24 jjw24 added the bug Something isn't working label Feb 13, 2021
@jjw24 jjw24 added this to the 1.7.1 milestone Feb 13, 2021
@jjw24
Copy link
Member

jjw24 commented Feb 13, 2021

Does this actually solve the problem in issue 86? As in will it actually improve the rendering?

@taooceros
Copy link
Member Author

taooceros commented Feb 13, 2021

Does this actually solve the problem in issue 86? As in will it actually improve the rendering?

The rendering won't be changed, but the ico file contains 16×16 icon so wpf don't need to downscaling the icon.
Well, if the 16×16 icon satisfying, it can be solved, but actually it isn't not that perfect.

@taooceros
Copy link
Member Author

taooceros commented Feb 13, 2021

After

image

Before

image

@jjw24 jjw24 merged commit 6a31868 into Flow-Launcher:dev Feb 13, 2021
@jjw24 jjw24 mentioned this pull request Feb 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working kind/ui related to UI, icons, themes, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants