Skip to content

[Feature] Take WinUI 2.x as a dependency to resolve XAML Style conflicts #3598

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

Closed
michael-hawker opened this issue Dec 3, 2020 · 2 comments
Labels
feature request 📬 A request for new changes to improve functionality improvements ✨ open discussion ☎️ WinUI 💠 Related to WinUI 3 Version or when paired with External can mean requires fix in WinUI 2/3.
Milestone

Comments

@michael-hawker
Copy link
Member

Describe the problem this feature would solve

Currently we try and base the Toolkit controls on top of the system included generic.xaml styles. This can potentially be hazardous as different OS versions contain different styles; however, they have been more stable as late. This is mostly due to all updates to styles being provided within the WinUI 2.x packages.

However, this now means that if someone includes both the Toolkit and WinUI 2.x in their project, the overridden default styles from WinUI 2 and conflict with the assumptions made for us building the Toolkit controls from the OS styles.

Describe the solution

If we take a hard dependency on WinUI 2 (which could help with the transition to WinUI 3 later anyway), then we can ensure that we target all our styles to build on top of the more stable set of styles included in the specific version of WinUI 2.x that we target. Then we can just tell developers to ensure they use the compatible WinUI 2 version and/or we ensure to ship updates when new WinUI 2 packages are shipped.

Describe alternatives you've considered

Trying to only target system styles or include extra copies of properties needed from updated styles.

Additional context & Screenshots

This would not be something we fix for 7.0 I believe. It's be a question if adding this dependency would require a major version rev and be a 'breaking change' or if we could do it in a 7.1/7.2 update?

@michael-hawker michael-hawker added open discussion ☎️ improvements ✨ feature request 📬 A request for new changes to improve functionality WinUI 💠 Related to WinUI 3 Version or when paired with External can mean requires fix in WinUI 2/3. labels Dec 3, 2020
@michael-hawker michael-hawker added this to the future milestone Dec 3, 2020
@ghost
Copy link

ghost commented Dec 3, 2020

Hello, 'michael-hawker! Thanks for submitting a new feature request. I've automatically added a vote 👍 reaction to help get things started. Other community members can vote to help us prioritize this feature in the future!

@michael-hawker
Copy link
Member Author

We did this in 7.0, closing.

@ghost ghost locked as resolved and limited conversation to collaborators Jan 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature request 📬 A request for new changes to improve functionality improvements ✨ open discussion ☎️ WinUI 💠 Related to WinUI 3 Version or when paired with External can mean requires fix in WinUI 2/3.
Projects
None yet
Development

No branches or pull requests

1 participant