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

Color > Hight Contrast: Incorrect Highlighting on Theme Toggle #1671

Closed
Zakariathr22 opened this issue Nov 29, 2024 · 5 comments · Fixed by #1680
Closed

Color > Hight Contrast: Incorrect Highlighting on Theme Toggle #1671

Zakariathr22 opened this issue Nov 29, 2024 · 5 comments · Fixed by #1680
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@Zakariathr22
Copy link
Contributor

Zakariathr22 commented Nov 29, 2024

Which version of the app?

WinUI 3 Gallery

Steps to Reproduce:

  1. Navigate to the "Color > Hight Contrast".
  2. Toggle the theme using the theme switch button at the top right corner.
  3. Observe the changes in the text highlighting.

Screenshots

Before Toggling the theme
Image

After Toggling the theme
Image

Expected Behavior:

The text should maintain consistent highlighting and text visibility regardless of the theme, with proper contrast between the background and text.

Actual Behavior:

After toggling the theme, the text become poorly highlighted, making it less readable and causing visual inconsistency.

@Zakariathr22 Zakariathr22 added bug Something isn't working needs-triage labels Nov 29, 2024
@Jay-o-Way
Copy link
Contributor

Got the same on pages: Iconography, Spacing, ColorPaletteResources, to name a few

@Zakariathr22
Copy link
Contributor Author

Got the same on pages: Iconography, Spacing, ColorPaletteResources, to name a few

The issue in 'SpacingPage' was resolved in #1639

@marcelwgn
Copy link
Contributor

Thanks for reporting that bug @Zakariathr22 ! In case you want to try and fix this, I think an easy solution would be remove the CodePresenter that is on that page since it does not make a lot sense in the high contrast section.

@Jay-o-Way The pages Spacing and Iconography were fixed by the PR @Zakariathr22 linked. ColorPaletteResources is indeed still an issue. If you find other pages that do not properly support the toggle, feel free to create issues for them!

@Jay-o-Way
Copy link
Contributor

Spacing and Iconography were fixed by the PR

That's interesting, because that PR only changes one file...

@marcelwgn
Copy link
Contributor

I guess the Iconography page was fixed by some other changes then. I think there were some changes to finding the element to change theme at some point but I cant remember the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
3 participants