maui icon indicating copy to clipboard operation
maui copied to clipboard

[regression/8.0.0-preview.1.7762] Border + RefreshView will let ScrollView no SizeChanged

Open xtuzy opened this issue 2 years ago • 4 comments
trafficstars

Description

When a Border in ScrollView and ScrollView in RefreshView, ScrollView no SizeChanged event. sharex-20230812075006

When i delete Border or Delete RefreshView, ScrollView have SizeChanged event. sharex-20230812075049

sharex-20230812075107

I find this bug when i test my project MauiUICollectionView on net8 preview7, when change window size, content don't fit window size.

Steps to Reproduce

  1. clone https://github.com/xtuzy/net8preview7_refeshviewborderbug
  2. change windows size, see output information

Link to public reproduction project repository

https://github.com/xtuzy/net8preview7_refeshviewborderbug

Version with bug

8.0.0-preview.7.8842

Is this a regression from previous behavior?

Yes, this used to work in .NET MAUI

Last version that worked well

7.0.92

Affected platforms

Android, Windows, I was not able test on other platforms

Affected platform versions

windows10.0.19041.0

Did you find any workaround?

No response

Relevant log output

No response

xtuzy avatar Aug 12 '23 00:08 xtuzy

Verified this on Visual Studio Enterprise 17.8.0 Preview 1.0. Repro on Windows 11 .NET 8 with below Project: 16711.zip

XamlTest avatar Aug 22 '23 08:08 XamlTest

I have another repro that I believe to be the same bug: https://github.com/Redth/MauiRepro_RefreshScrollRepro in my case it seems largely problematic on Mac/iOS

Redth avatar Aug 22 '23 14:08 Redth

Confirmed that this regressed between https://github.com/dotnet/maui/compare/8.0.0-preview.1.7762..7.0.92. https://github.com/dotnet/maui/pull/11357 seems sus.

samhouts avatar Sep 08 '23 00:09 samhouts

image Where is the Border in the visual tree? Inside the ScrollView? I have added several Borders and with the latest version the SizeChanged event of the ScrollView is triggered.

jsuarezruiz avatar Nov 06 '23 08:11 jsuarezruiz