-
Notifications
You must be signed in to change notification settings - Fork 707
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(binding): Avoid reading source property on UpdateSource for non-DP #13698
Conversation
The build 94711 found UI Test snapshots differences: Details
|
The build 94783 found UI Test snapshots differences: Details
|
@Mergifyio backport release/stable/4.9 release/stable/4.10 legacy/4x |
✅ Backports have been created
|
GitHub Issue (If applicable): closes #11956
PR Type
What kind of change does this PR introduce?
What is the new behavior?
During a two-way binding, the source bound property will not be read before being set. This will avoid scenarios during datacontext reset that may read properties for out of date instances.
Copilot Summary
🤖 Generated by Copilot at 203609d
This pull request enhances the data binding system and the
ComboBox
control in Uno.UI. It adds a new unit test forComboBox
, refactors theBindingItem
class to use flags and support a new scenario, fixes a data binding bug in theBindingPath
class, and improves the performance and memory usage of the binding system.PR Checklist
Please check if your PR fulfills the following requirements:
Screenshots Compare Test Run
results.Other information
Internal Issue (If applicable):