ThemeService Documentation outdated?

[DevTKSS]

On which page?

https://platform.uno/docs/articles/external/uno.extensions/doc/Learn/ThemeService/HowTo-UseThemeService.html

What's wrong?

Normally when we add Features at the project creation, in this case ThemeService, its implemented already in the app.xaml.cs file, and we find the host config already existing. Seems to not be the case for UseThemeSwitching?


See the unchanged features in the csproj and the setup of the host config (also not changed!)
But somehow I did have the .GetThemeService() on the Pages this.GetThemeService().SetThemeAsync(AppTheme.Dark); Maybe you could check if that config is missing in template or not longer needed which would be a explaination for its not added from the wizard?

Any feedback?

Maybe you could add the way you call this task also to the documentation page, like if you do it with command via command extensions (toggleSwitch is not having Command by default) that could also add some improvement and help understand usage of this extension for controls which are not having this in a kind of usecase :)

[morning4coffe-dev]

@DevTKSS Thank you for reporting this issue! I have managed to confirm this behavior, I have created a new project and tried just "Consume ThemeService" and it works 🤔 I will try to get more details and adjust the docs.

[DevTKSS]

Correct i also got theme service add-able in e.g. pages codebehind without adding the thing in app.xaml.cs file. To the fact of "ToggleSwitch" is not able to bind command to do the theme service toggle task written down there, I would prefer to rename the task for reducing the risk of newbes do not see this and try while ToggleSwitch is not supported for commands like that.