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

fix: TabView does not render item content with bindings #4896

Merged
merged 8 commits into from
Jan 20, 2021

Conversation

MartinZikmund
Copy link
Member

@MartinZikmund MartinZikmund commented Jan 13, 2021

GitHub Issue (If applicable): fixes #4895, fixes #4899, fixes #4875, fixes #4907

PR Type

What kind of change does this PR introduce?

  • Bugfix

What is the current behavior?

Content is not rendered.

What is the new behavior?

Content is rendered.

PR Checklist

Please check if your PR fulfills the following requirements:

@MartinZikmund MartinZikmund requested a review from a team January 13, 2021 19:45
@gitpod-io
Copy link

gitpod-io bot commented Jan 13, 2021

@MartinZikmund MartinZikmund self-assigned this Jan 13, 2021
@MartinZikmund MartinZikmund added the ready-to-merge Automatically merge the PR once all '.mergify.yml' policies are met label Jan 16, 2021
@MartinZikmund MartinZikmund force-pushed the dev/mazi/tabview-databind branch 2 times, most recently from dcda9f1 to 75a3762 Compare January 17, 2021 16:19
@MartinZikmund MartinZikmund removed the ready-to-merge Automatically merge the PR once all '.mergify.yml' policies are met label Jan 18, 2021
- Test page to verify tab content changes when switching between data-bound items
- Workaround for unoplatform#4894 in TabView - sets the ContentPresenter's DataContext so that it matches the DataContext of the TabViewItem.
- Fixes issues unoplatform#4899 and unoplatform#4907
- The applied Uno specific workaround is reported as issue unoplatform#4925
@MartinZikmund MartinZikmund force-pushed the dev/mazi/tabview-databind branch from eb4a2ce to ef8e6bc Compare January 19, 2021 23:12
@MartinZikmund MartinZikmund added the ready-to-merge Automatically merge the PR once all '.mergify.yml' policies are met label Jan 19, 2021
@mergify mergify bot merged commit 9c6b8d2 into unoplatform:master Jan 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Automatically merge the PR once all '.mergify.yml' policies are met
Projects
None yet
3 participants