Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Regression Bug. Nested ScrollView does not work in Android. Works in iOS. #20920

Closed
Manish-Pradhan-FP opened this issue Feb 29, 2024 · 1 comment · Fixed by #20998
Closed

Regression Bug. Nested ScrollView does not work in Android. Works in iOS. #20920

Manish-Pradhan-FP opened this issue Feb 29, 2024 · 1 comment · Fixed by #20998
Assignees
@PureWeen
Labels
area-controls-scrollview ScrollView fixed-in-8.0.10 fixed-in-9.0.0-preview.3.10457 i/regression This issue described a confirmed regression on a currently supported version platform/android 🤖 t/bug Something isn't working
Milestone
.NET 8 SR3

Comments

@Manish-Pradhan-FP
Copy link

Manish-Pradhan-FP commented Feb 29, 2024 •
edited by PureWeen
Loading

Description

More details here #7811
This bug existed in DotNet 7 and was fixed in DotNet 8 but in its back in 8.0.7. If I revert back to 8.0.6 the issue goes away.

Steps to Reproduce

<ScrollView Orientation="Horizontal">
    <ScrollView Orientation="Vertical"/>
</Scrollview>

The inner scrollview does not scroll in Android.

Link to public reproduction project repository

No response

Version with bug

8.0.7 SR2

Is this a regression from previous behavior?

Yes, this used to work in .NET MAUI

Last version that worked well

8.0.6 SR1
8.0.4-nightly.9742 (I realize 4 is less then 6 but this version is newer than 8.0.6)

Affected platforms

Android

Affected platform versions

No response

Did you find any workaround?

No response

Relevant log output

No response
@Manish-Pradhan-FP Manish-Pradhan-FP added the t/bug Something isn't working label Feb 29, 2024
@jsuarezruiz jsuarezruiz added area-controls-scrollview ScrollView potential-regression This issue described a possible regression on a currently supported version., verification pending platform/android 🤖 labels Feb 29, 2024
@PureWeen PureWeen added i/regression This issue described a confirmed regression on a currently supported version and removed potential-regression This issue described a possible regression on a currently supported version., verification pending labels Feb 29, 2024
@PureWeen PureWeen added this to the .NET 8 SR3 milestone Feb 29, 2024
@PureWeen
Copy link
Member

My current guess is that this behavior changed here

#19349

@PureWeen PureWeen moved this from Todo to In Progress in MAUI SDK Ongoing Mar 4, 2024
@PureWeen PureWeen mentioned this issue Mar 4, 2024
Merged
@PureWeen PureWeen self-assigned this Mar 4, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in MAUI SDK Ongoing Mar 5, 2024
@github-actions github-actions bot mentioned this issue May 3, 2024
Open
@github-actions github-actions bot locked and limited conversation to collaborators May 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@PureWeen PureWeen

Labels
area-controls-scrollview ScrollView fixed-in-8.0.10 fixed-in-9.0.0-preview.3.10457 i/regression This issue described a confirmed regression on a currently supported version platform/android 🤖 t/bug Something isn't working
Projects
MAUI SDK Ongoing
Status: Done
Milestone
.NET 8 SR3
Development

Successfully merging a pull request may close this issue.

Don't remeasure the HScrollView during the layout pass dotnet/maui
4 participants
@PureWeen @jsuarezruiz @samhouts @Manish-Pradhan-FP