Skip to content

Commit

Permalink
fix(itemsreapeter): Workarround only first item rendered if no Scroll…
Browse files Browse the repository at this point in the history
…Viewer
  • Loading branch information
dr1rrb committed Feb 2, 2021
1 parent a3fee07 commit bd3265a
Showing 1 changed file with 8 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -501,13 +501,20 @@ void EnsureScroller()
parent = CachedVisualTreeHelpers.GetParent(parent);
}

/*
Uno workaround:
This a workaround for https://github.com/microsoft/microsoft-ui-xaml/issues/2349.
While this bugs occurs only for `Popup` on WinUI, it might occurs even in the "main" visual tree on uno.
The reason is that they do have a `ScrollViewer` at the root of the app on WinUI (to support the SIP), but we don't in Uno.
if (m_scroller == null)
{
// We usually update the viewport in the post arrange handler. But, since we don't have
// a scroller, let's do it now.
UpdateViewport(new Rect( ));
}
else if (!m_managingViewportDisabled)
else*/
if (!m_managingViewportDisabled)
{
m_effectiveViewportChangedRevoker = Disposable.Create(() =>
{
Expand Down

0 comments on commit bd3265a

Please sign in to comment.