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

Setter Specificity #11082

Closed
Tracked by #14032
StephaneDelcroix opened this issue Nov 3, 2022 · 1 comment · Fixed by #13818
Closed
Tracked by #14032

Setter Specificity #11082

StephaneDelcroix opened this issue Nov 3, 2022 · 1 comment · Fixed by #13818
Assignees
Labels
area-xaml XAML, CSS, Triggers, Behaviors fixed-in-8.0.0-preview.7.8842 Look for this fix in 8.0.0-preview.7.8842! partner/hot-reload-xaml Issues impacting XAML Hot Reload experiences proposal/open t/enhancement ☀️ New feature or request
Milestone

Comments

@StephaneDelcroix
Copy link
Contributor

Description

In order to better compute the current values for BindableProperties, we need to keep a list of all values being set, with their Specificity, instead of relying on some heuristics (from style, etc.) and order of application.

The specificity concept is borrowed from CSS, and keeping a list of values with specificity will allow

  • finer grained application of value
  • more robust unapply of values (think VSM, or Styles)
  • better support for HotReload

(Public) API Changes

No public API change

Usage Scenarios

/

Backward Compatibility

This will fix a few existing bugs, so might break apps relying on those bugs or wrong behaviours. We're thinking about walling this behind a feature flag, enabled by default on the new templates, opening the opportunity to include this as a service release of net7

Difficulty

Medium

@Redth Redth added the partner/hot-reload-xaml Issues impacting XAML Hot Reload experiences label Nov 3, 2022
@Redth Redth moved this to In Progress in MAUI SDK Ongoing Nov 3, 2022
@PureWeen PureWeen added this to the Backlog milestone Nov 3, 2022
@ghost
Copy link

ghost commented Nov 3, 2022

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-xaml XAML, CSS, Triggers, Behaviors fixed-in-8.0.0-preview.7.8842 Look for this fix in 8.0.0-preview.7.8842! partner/hot-reload-xaml Issues impacting XAML Hot Reload experiences proposal/open t/enhancement ☀️ New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants