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

[ToDo][Win/iOS][Settings Flyout] Opening Settings Flyout Changes App Theme Without Interacting with Light/Dark Theme Controls #778

Open
2 of 4 tasks
agneszitte opened this issue Jul 9, 2024 · 1 comment · May be fixed by #783
Assignees

Comments

@agneszitte
Copy link
Contributor

agneszitte commented Jul 9, 2024

Current behavior

Opening the settings flyout changes the theme of the app without having to click on the Light/Dark theme chips controls in the flyout.

Windows:

341e849f1132fd8f5eb454d20f8fbba6.mp4

iOS:

06ab93685fcbe14cdeab8c15dab1fdae.mp4

Expected behavior

Opening the settings flyout should not change the theme of the app unless the user interacts with the Light/Dark theme chips controls.

How to reproduce it (as minimally and precisely as possible)

  • Open the application in Wide Layout.
  • Click on the "Get Started" button on the Welcome Page.
  • Click on the user name button in the top left corner of the app at the top of the menu to open the settings flyout.
  • Observe that the theme of the app changes automatically without clicking on the Light/Dark theme chips controls.
    (Do multiple retries to notice the issue if needed)

Environment

Nuget Package:

  • Uno.Sdk: 5.2.161
  • UnoExtensionsVersion: 4.1.28
  • UnoToolkitVersion: 6.0.24
  • UnoThemesVersion: 5.0.13

Affected platform(s):

  • Windows
  • iOS
  • Android
  •  WebAssembly

Additional information

Possible related issue: #780

@agneszitte
Copy link
Contributor Author

agneszitte commented Jul 9, 2024

@rajamatt / @kazo0
The current logic will need to be reviewed:

ItemChecked="ThemeChipGroup_ItemChecked"

// Workaround for https://github.com/unoplatform/uno/issues/15287

For Windows every time you open the Settings Flyout you go through here if I am not mistaken:

void IRecipient<ThemeChangedMessage>.Receive(ThemeChangedMessage message)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants