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 overlapping items in VirtualizingStackPanel
#13765
Conversation
This ports the fix in AvaloniaUI/Avalonia.Controls.TreeDataGrid#229 to Avalonia. Fixes #12744
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.
Tested against the sample from #12744. LGTM!
I didn't expect a one line fix, nice :)
Awesome! Thank you very much! |
probably, yes. At least it's planned. Will be available in nightly and 11.1 for sure. So if you need this fix faster, try nightly for now. |
* Added failing test for #12744. * Inserting items makes the StartU unstable. This ports the fix in AvaloniaUI/Avalonia.Controls.TreeDataGrid#229 to Avalonia. Fixes #12744 #Conflicts: # tests/Avalonia.Controls.UnitTests/VirtualizingStackPanelTests.cs
Is there some kind of release schedule? |
As you can see, this PR has the There also seems to be a new git tag for 11.0.6, so it may be released soon. The 11.0.6 packages appear in the nightly feed, if you want to test. |
What does the pull request do?
As described in #12744, sometimes items in a virtualized
ItemsControl
would start overlapping.The reason is simple: inserting items before the current viewport should cause
RealizedStackElements
to markStartU
as unstable because we don't now actually know the start U position any more (we haven't measured the inserted item).This is a port of a fix for
TreeDataGrid
but this time I actually wrote a simple unit test which tests the source of the problem (although a unit test for the actual visible effect of this bug is a lot more difficult, and I've still not managed to repro in a unit test).Fixed issues
Fixes #12744
Part of #13736