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

ScrollView with Fill vertical option is bigger than the page when ScrollView is a root on iOS #18975

Closed
cucumber-sp opened this issue Nov 23, 2023 · 2 comments
Labels
area-controls-scrollview ScrollView platform/iOS 🍎 potential-regression This issue described a possible regression on a currently supported version., verification pending s/needs-repro Attach a solution or code which reproduces the issue t/bug Something isn't working

Comments

@cucumber-sp
Copy link

cucumber-sp commented Nov 23, 2023

Description

When ScrollView (or ScrollView inside RefreshView) is a root of Content page, it doesn't handle Height correctly. When you set VerticalOptions to Fill (when inside RefreshView it's always Fill) it takes more space than the page size. When you set Start option it makes ScrollView even smaller than its content.

image
VerticalOptions="Fill" (content of scrollView starts at the title, it's not a padding)

image
VerticalOptions="Start"

image
Grid is the root of page, everything works fine

Steps to Reproduce

No response

Link to public reproduction project repository

No response

Version with bug

8.0.3

Is this a regression from previous behavior?

Yes, this used to work in .NET MAUI

Last version that worked well

Unknown/Other

Affected platforms

iOS

Affected platform versions

No response

Did you find any workaround?

You can make Grid the root of the page and put ScrollView inside, then it will have correct size

Relevant log output

No response

@cucumber-sp cucumber-sp added the t/bug Something isn't working label Nov 23, 2023
@jsuarezruiz jsuarezruiz added area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter platform/iOS 🍎 area-controls-scrollview ScrollView potential-regression This issue described a possible regression on a currently supported version., verification pending labels Nov 23, 2023
@jsuarezruiz jsuarezruiz added the s/needs-repro Attach a solution or code which reproduces the issue label Dec 4, 2023
@ghost
Copy link

ghost commented Dec 4, 2023

Hi @cucumber-sp. We have added the "s/needs-repro" label to this issue, which indicates that we require steps and sample code to reproduce the issue before we can take further action. Please try to create a minimal sample project/solution or code samples which reproduce the issue, ideally as a GitHub repo that we can clone. See more details about creating repros here: https://github.com/dotnet/maui/blob/main/.github/repro.md

This issue will be closed automatically in 7 days if we do not hear back from you by then - please feel free to re-open it if you come back to this issue after that time.

@ghost ghost added the s/no-recent-activity Issue has had no recent activity label Dec 8, 2023
@ghost
Copy link

ghost commented Dec 8, 2023

This issue has been automatically marked as stale because it has been marked as requiring author feedback to reproduce the issue but has not had any activity for 4 days. It will be closed if no further activity occurs within 3 days of this comment. If it is closed, feel free to comment when you are able to provide the additional information and we will re-investigate.

@ghost ghost closed this as completed Dec 11, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Jan 11, 2024
@ghost ghost removed the s/no-recent-activity Issue has had no recent activity label Jan 11, 2024
@Eilon Eilon removed the area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter label May 14, 2024
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-controls-scrollview ScrollView platform/iOS 🍎 potential-regression This issue described a possible regression on a currently supported version., verification pending s/needs-repro Attach a solution or code which reproduces the issue t/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants