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.

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

[9.0 GA, Android] ScrollView starts at the position of first Entry control on the bottom rather than at 0 #25829

Closed
janne-hmp opened this issue Nov 13, 2024 · 9 comments · Fixed by #26101
Labels
area-controls-scrollview ScrollView fixed-in-9.0.21 i/regression This issue described a confirmed regression on a currently supported version platform/android 🤖 s/needs-attention Issue has more information and needs another look s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Milestone

Comments

@janne-hmp
Copy link

janne-hmp commented Nov 13, 2024

Description

In .NET MAUI 9.0 GA, Android has a new annoying feature that ScrollView does not start at position 0 but at the position where the content's first Entry control is located on the bottom of the ScrollView. I have a long list of settings on the app's Settings page and it is very confusing if the page is opened in the middle of the settings list rather than at start.

Windows seems to work fine, didn't test iOS. This also worked fine on Android in .NET MAUI 9.0 RC2.

Steps to Reproduce

  1. Download and build GnollHackMAUIEasyBuild for Android by following the given instructions
  2. Start the app and press the Settings button
  3. Settings are opened in the middle of the list with User Name entry being on the bottom

Link to public reproduction project repository

https://github.com/hyvanmielenpelit/GnollHackMAUIEasyBuild

Version with bug

9.0.0 GA

Is this a regression from previous behavior?

Yes, this used to work in .NET MAUI

Last version that worked well

9.0.0-rc.2.24503.2

Affected platforms

Android

Affected platform versions

Android 14

Did you find any workaround?

Disabling all Entries at start and then enabling them with a 0.5 sec one-off timer seemed to help, but is not very elegant.

Relevant log output

@janne-hmp janne-hmp added the t/bug Something isn't working label Nov 13, 2024
@PureWeen PureWeen added potential-regression This issue described a possible regression on a currently supported version., verification pending partner/macios Issues for the Mac / iOS SDK area-controls-scrollview ScrollView labels Nov 13, 2024
@ninachen03 ninachen03 added s/verified Verified / Reproducible Issue ready for Engineering Triage s/triaged Issue has been reviewed platform/android 🤖 and removed potential-regression This issue described a possible regression on a currently supported version., verification pending labels Nov 15, 2024
@ninachen03
Copy link

This issue has been verified using Visual Studio 17.13.0 Preview 1.0 (9.0.0 & 9.0.0-rc.2.24503.2). Can repro on Android platform.

@janne-hmp janne-hmp changed the title [9.0 GA, Android] ScrollView starts at the position of first Entry control at start rather than at 0 [9.0 GA, Android] ScrollView starts at the position of first Entry control on the bottom rather than at 0 Nov 15, 2024
@mattleibow
Copy link
Member

@ninachen03 did this happen in .NET 8? What about net9 preview6 or 7?

@mattleibow mattleibow added this to the Backlog milestone Nov 18, 2024
@mattleibow mattleibow added potential-regression This issue described a possible regression on a currently supported version., verification pending and removed partner/macios Issues for the Mac / iOS SDK labels Nov 18, 2024
@mattleibow mattleibow modified the milestones: Backlog, .NET 9 Servicing Nov 18, 2024
@PureWeen
Copy link
Member

@janne-hmp can you provide a simplified sample that reproduces the behavior?

Archive.zip

I did a quick test here with entries in a scrollview and it's not scrolling to the first entry automatically.

@PureWeen PureWeen added the s/needs-info Issue needs more info from the author label Nov 18, 2024
@ninachen03
Copy link

ninachen03 commented Nov 19, 2024

@ninachen03 did this happen in .NET 8? What about net9 preview6 or 7?

Sorry, let me add some clarity,
This issue is reproduced on 9.0.0-rc.2.24503.2, but not on 9.0.0-rc.1.24453.9 to 9.0.0-preview.1.9973, so it is a regression issue.
Image

@dotnet-policy-service dotnet-policy-service bot added s/needs-attention Issue has more information and needs another look and removed s/needs-info Issue needs more info from the author labels Nov 19, 2024
@ninachen03 ninachen03 added s/needs-info Issue needs more info from the author i/regression This issue described a confirmed regression on a currently supported version and removed s/needs-attention Issue has more information and needs another look potential-regression This issue described a possible regression on a currently supported version., verification pending labels Nov 19, 2024
@janne-hmp
Copy link
Author

Hi, the game works just on .NET MAUI 9 (and Xamarin) currently. As Nina Chen mentioned, the problem appears on Android just with 9.0 GA, not with 9.0 RC2. Xamarin works well, too. I cannot check .NET 8 or 7 at the moment, as a meaningful amount of additional work would potentially be needed to add the support for older framework versions.

@dotnet-policy-service dotnet-policy-service bot added s/needs-attention Issue has more information and needs another look and removed s/needs-info Issue needs more info from the author labels Nov 19, 2024
@ninachen03
Copy link

Hi, the game works just on .NET MAUI 9 (and Xamarin) currently. As Nina Chen mentioned, the problem appears on Android just with 9.0 GA, not with 9.0 RC2. Xamarin works well, too. I cannot check .NET 8 or 7 at the moment, as a meaningful amount of additional work would potentially be needed to add the support for older framework versions.

I used the provided repro on RC2, the problem reappeared. It worked fine on RC1.

@janne-hmp
Copy link
Author

Thanks for the clarification, Nina. I probably did not notice the bug in RC2, so good to hear the regression has been found to have occurred already in RC2 instead, and RC1 is the version which works.

@PureWeen
Copy link
Member

Fairly sure this was caused by

#24064

@jsuarezruiz
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-controls-scrollview ScrollView fixed-in-9.0.21 i/regression This issue described a confirmed regression on a currently supported version platform/android 🤖 s/needs-attention Issue has more information and needs another look s/triaged Issue has been reviewed s/verified Verified / Reproducible Issue ready for Engineering Triage t/bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

6 participants