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

[X] inherit DataType based on attribute #25173

Merged
merged 1 commit into from
Oct 16, 2024
Merged

[X] inherit DataType based on attribute #25173

merged 1 commit into from
Oct 16, 2024

Conversation

StephaneDelcroix
Copy link
Contributor

@StephaneDelcroix StephaneDelcroix commented Oct 10, 2024

Description of Change

Changes the heuristic for deciding if we need ot ignore parent DataType, as DataTrigger.Binding shoud inherit it, and
Picker.ItemDisplayNameBinding should not

Issues Fixed

this is an alternate fix for #23989, partially replaces #24513, and will help fixing #25141 together with #25152

Change the heuristic for deciding if we need ot ignore parent DataType,
as DataTrigger.Binding shoud inherit it, and
Picker.ItemDisplayNameBinding should not

this is an alternate fix for #23989, partially replaces #24513, and will
help fixing #25141 together with #24152
@StephaneDelcroix StephaneDelcroix requested a review from a team as a code owner October 10, 2024 10:11
@PureWeen PureWeen added this to the .NET 9.0 GA milestone Oct 10, 2024
@@ -0,0 +1,9 @@
#nullable disable
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nitpick: could probably remove this line, but don't block merging. 👍

@PureWeen
Copy link
Member

/backport to release/9.0.1xx

Copy link
Contributor

Started backporting to release/9.0.1xx: https://github.com/dotnet/maui/actions/runs/11352810934

@PureWeen PureWeen merged commit 028cd0c into net9.0 Oct 16, 2024
121 checks passed
@PureWeen PureWeen deleted the fix_23989_again branch October 16, 2024 17:46
@github-actions github-actions bot locked and limited conversation to collaborators Nov 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants