Skip to content
This repository has been archived by the owner on May 3, 2024. It is now read-only.

Icon Consistency #3

Closed
jmacato opened this issue Oct 14, 2019 · 5 comments
Closed

Icon Consistency #3

jmacato opened this issue Oct 14, 2019 · 5 comments
Labels

Comments

@jmacato
Copy link
Contributor

jmacato commented Oct 14, 2019

Since we have adopted a Visual Studio-style User Interface, it is imperative that we have to sort out our Icon Design Language to be more inline with the guidelines as shown here:
https://docs.microsoft.com/en-us/visualstudio/extensibility/ux-guidelines/images-and-icons-for-visual-studio?view=vs-2019

@jmacato jmacato added the enhancement New feature or request label Oct 14, 2019
@molnard
Copy link
Contributor

molnard commented Oct 15, 2019

If we are planning to use design guidelines: #6
We will have icons like this: https://material.io/resources/icons/?style=baseline

@jmacato
Copy link
Contributor Author

jmacato commented Mar 9, 2020

For reference: Comparisons of VS's handling of Solution Explorer Icons

Light Theme, 9pt font size:
image

Dark Theme, 9pt:
image

Light Theme, 15pt:
image

Dark Theme, 15pt:
image

Observations:

  1. They dont scale the icons with the font, we could do better than that since we use vectorized icons everywhere anyway.

  2. The grayscale palette described in the VS Icon design is inverted when it switches to dark mode. Accent colors are ignored.

@molnard
Copy link
Contributor

molnard commented Mar 10, 2020

I like it and also that the user will have an option to use light mode.

@jmacato
Copy link
Contributor Author

jmacato commented Mar 11, 2020

AvaloniaUI/Avalonia#3655 prevents us from doing dynamic color change for icons and other controls that are declared inside styles

Seems to be fixed at Avalonia master branch. Though the fix is not backportable so the color/theme change feature may wait until the next major release...

@jmacato
Copy link
Contributor Author

jmacato commented Mar 23, 2020

Addressed by WalletWasabi/WalletWasabi#3267 :)

@jmacato jmacato closed this as completed Mar 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants