-
Notifications
You must be signed in to change notification settings - Fork 743
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
chore: ITemplatedParentProvider removal #18443
chore: ITemplatedParentProvider removal #18443
Conversation
b85c2d0
to
32f326d
Compare
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-18443/index.html |
🤖 Your WebAssembly Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-18443/index.html |
....UI.SourceGenerators.Tests/DependencyObjectGeneratorTests/Given_DependencyObjectGenerator.cs
Show resolved
Hide resolved
src/SourceGenerators/Uno.UI.SourceGenerators/DependencyObject/DependencyObjectGenerator.cs
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs to be tested at least on Gallery just to make sure it's not breaking the legacy impl. But changes LGTM as long as it will be backported to 5.5 soon.
32f326d
to
684bf20
Compare
@Mergifyio backport release/stable/5.5 |
✅ Backports have been created
|
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-18443/index.html |
684bf20
to
ba8410f
Compare
ba8410f
to
147c879
Compare
🤖 Your WebAssembly Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-18443/index.html |
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-18443/index.html |
🤖 Your WebAssembly Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-18443/index.html |
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-18443/index.html |
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-18443/index.html |
🤖 Your WebAssembly Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-18443/index.html |
🤖 Your WebAssembly Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-18443/index.html |
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-18443/index.html |
The build 143346 found UI Test snapshots differences: Details
|
…5.5/pr-18443 chore: ITemplatedParentProvider removal (backport #18443)
GitHub Issue (If applicable): n/a
PR Type
What kind of change does this PR introduce?
What is the new behavior?
follow-up of #17645
removing ITemplatedParentProvider and relevant codegen in DOGenerator, replaced with direct implementations in DOStore.
add a few more tests against regressions found in the tp rework.
ninja-fix: native Frame not being initialized.
PR Checklist
Please check if your PR fulfills the following requirements:
Screenshots Compare Test Run
results.Other information
With changes from this PR and #17645, there are a few notable changes to the templated-parent system:
DO::TemplatedParent
dependency property,::OnTemplatedParentChanged(e)
, andDOStore.SetTemplatedParent(FrameworkElement)
are fully deprecated. They are no longer called by the Uno framework, and should not be used anymore. They are, however, still temporarily kept in v5.5.0 to avoid binary changes until next major version.OnTemplatedParentChanged
, should be replaced byOnLoaded()
(within uno.ui assembly) orFE.Loaded += ...
(outside uno).