Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

RelativeSource Binding Bug Fixes (Gh8450) #8541

Merged
merged 2 commits into from
Nov 22, 2019

Conversation

legistek
Copy link
Contributor

@legistek legistek commented Nov 17, 2019

Description of Change

See #8450 which describes two issues left over from the original inclusion of the feature in #4375

Issues Resolved

Fixes #8450

API Changes

None

Platforms Affected

All

Behavioral/Visual Changes

FindAncestorBindingContext mode will function correctly with BindingContext changes and reparenting.
AncestorLevel will function in accordance with the documentation.

Testing Procedure

Unit tests for this feature have been revised.

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

@legistek legistek changed the title RelativeSource Binding Bug Fixes (Gh7459) RelativeSource Binding Bug Fixes (Gh8450) Nov 17, 2019
Copy link
Member

@StephaneDelcroix StephaneDelcroix left a comment

Choose a reason for hiding this comment

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

please retarget to 4.4.0

@StephaneDelcroix StephaneDelcroix added the retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. label Nov 18, 2019
(1) Fix RelativeSource-FindAncestorBindingContext bindings not re-applying on ancestor BindingContext or ancestry changes; 
(2) fix incorrect interpretation of AncestorLevel; 
(3) much more robust unit tests for RelativeSource ancestor-type binding (#4)
@legistek legistek force-pushed the gh8450-relsourcebugs-on-430 branch from 3e9f344 to 3e0cffb Compare November 18, 2019 13:55
@legistek legistek changed the base branch from 4.3.0 to 4.4.0 November 18, 2019 13:56
@legistek
Copy link
Contributor Author

Sure thing - done.

@samhouts samhouts removed the retarget-branch-required PR or associated issues target a milestone. Please target this PR to the matching branch. label Nov 18, 2019
Co-Authored-By: Samantha Houts <samhouts@users.noreply.github.com>
@samhouts samhouts self-assigned this Nov 19, 2019
@samhouts samhouts added the approved Has two approvals, no pending reviews, and no changes requested label Nov 20, 2019
@samhouts samhouts merged commit 16cdd28 into xamarin:4.4.0 Nov 22, 2019
@samhouts samhouts added this to the 4.4.0 milestone Nov 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a/Xaml </> approved Has two approvals, no pending reviews, and no changes requested Core t/bug 🐛
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants