-
Notifications
You must be signed in to change notification settings - Fork 697
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
fix(ResourceDictionary): Default theme may not be valid on first call #4857
Conversation
There are no tests for this specific change as it impacts the Application.Current. The validation for this change can be done through the Uno Gallery at unoplatform/Uno.Gallery@9fbf149. The background should be dark when system settings are dark.
The build 23348 found UI Test snapshots differences: Details
|
The build 23348 found UI Test snapshots differences: Details
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's no associated issue?
It's a fix on the spot from the canaries. |
GitHub Issue (If applicable): #
PR Type
What kind of change does this PR introduce?
What is the current behavior?
The first resolution of
ThemeDictionary
may return the light theme incorrectly.What is the new behavior?
There are no tests for this specific change as it impacts the
Application.Current
.The validation for this change can be done through the Uno Gallery at unoplatform/Uno.Gallery@9fbf149. The background should be dark when system settings are dark.
PR Checklist
Please check if your PR fulfills the following requirements:
Screenshots Compare Test Run
results.Other information
Internal Issue (If applicable):