-
Notifications
You must be signed in to change notification settings - Fork 33
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
Outdated and Duplicated Material Pages #1514
Comments
Hello @DevTKSS! Thanks for the report! So there is actually a valid case for both of these scenarios. The usage of Otherwise, if you are using the Uno.Toolkit.WinUI.Material library, you can initialize the Toolkit AND Material resources in a single line with Let me know if that clears things up or if there are other questions that I may not have covered! |
Honestly? No, not really... These both pages in documentation are looking almost the same and the difference if there is one, is not getting clear of any of them, sorry... Just if I would try to logical search for things like material and other theme related stuff, that would first be something named "Themes" If I would then search for things like control extensions and other enhancements not directly related to the theme thing, that would be something like toolkit. BUT there is that point "Wait, the controlextensions are using themes like the elevation and adding commands (if I overflowed that correctly yesterday) so they are something in-between..." And just as fun fact by the way: But that's only my own opinion, that does not mean everyone else has to agree with that! |
On which page?
Uno public available (html) Documentation enterable from Themes/Overview/Material:
Themes Namespace material-getting-started
since Themes Namespace material-migration is having same/similar namespaces, consider checking them also
What's wrong?
(Themes Docs Pages)
Currenty it's told to use the Uno.Material set like this:
and this for dsp usage:
But from wizard created using Material theme the currently created single head uno project app.xaml is including this:
I tryed doing as told there (uno.sdk 5.5.56) and with this in place the app is not building any longer because the resources could not be resolved and resulted in 'Exception_WasThrown' in the Programm.cs builder method on Desktop target
resetting it to the namspaces showed in last screenshot is getting no issue, only not showing any ResourceKeys to any Material style/theme and also not the one defined in the dsp!
Any feedback?
Duplicated Page?
Uno public available Documentation enterable from Toolkit/Overview/Material:
https://github.com/unoplatform/uno.toolkit.ui/blob/main/doc/material-getting-started.md/#L1
seems to be more correct (newer Version?) but also not representing what we get in the currently wizard generated single head projects
at least from my point this could work because correct Namespaces
this Note on the Toolkit section version made me (using xaml not c#) think about "am I missing a AppBuilder Extension Method? because in my wizard created app.xaml.cs file is none of this Methods. Only one including something with Toolkit is the
.UseToolkitNavigation()
for the Navigatorif you are editing maybe, could you add a bit text to tell also for the xaml users if that extension method have to be used or not? currently its not set up by default but seems to work beside that missing ResourceKeys in intellisence at designtime.
The text was updated successfully, but these errors were encountered: