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

IOS: Scrollview inside Refreshview #12132

Closed
FM1973 opened this issue Dec 15, 2022 · 6 comments · Fixed by #11357
Closed

IOS: Scrollview inside Refreshview #12132

FM1973 opened this issue Dec 15, 2022 · 6 comments · Fixed by #11357
Assignees
Labels
area-controls-refreshview RefreshView area-controls-scrollview ScrollView fixed-in-7.0.52 Look for this fix in 7.0.52 SR1.1! fixed-in-7.0.100 fixed-in-7.0.101 fixed-in-8.0.0-preview.1.7762 Look for this fix in 8.0.0-preview.1.7762! platform/iOS 🍎 t/bug Something isn't working

Comments

@FM1973
Copy link

FM1973 commented Dec 15, 2022

Description

Although the last update should have provided a bugfix - this is not the case.
When there is a scrollview inside a refreshview, you can´t scroll to the bottom.
In my sample there is height-request set for the collectionviews because of the height-bug (infinite height) of the collectionview in IOS.

Steps to Reproduce

  1. Clone repository
  2. Start the app
  3. Click on "Scrollview inside refreshview IOS"
  4. Try to scroll to the bottom

Link to public reproduction project repository

https://github.com/FM1973/RefreshGridRepo.git

Version with bug

7.0 (current)

Last version that worked well

Unknown/Other

Affected platforms

iOS

Affected platform versions

IOS 14 and up

Did you find any workaround?

Not really.

Relevant log output

No response

@FM1973 FM1973 added the t/bug Something isn't working label Dec 15, 2022
@FM1973 FM1973 changed the title ISO: Scrollview inside Refrehview IOS: Scrollview inside Refrehview Dec 15, 2022
@Eilon Eilon added the area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter label Dec 15, 2022
@jsuarezruiz jsuarezruiz added this to the Backlog milestone Dec 19, 2022
@ghost ghost added the legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor label Dec 19, 2022
@ghost
Copy link

ghost commented Dec 19, 2022

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

@ajay-mundi
Copy link

Is there a workaround for this issue?

@FM1973
Copy link
Author

FM1973 commented Dec 20, 2022

None I know of.

@FM1973
Copy link
Author

FM1973 commented Dec 20, 2022

Sadly this workaround #8820 does not work with a refreshview.

@FM1973
Copy link
Author

FM1973 commented Dec 20, 2022

For IOS only one can emulate the RefreshView effect.
Remove the refreshview. Then bind the scrolled event of the scrollview and start refreshing the viewmodel if the y position of the scrollview is i.e. smaller than -100. Use a bool value to stop handling the scrolled event while refreshing.
Because the scrollview in IOS has a bouncing effect, the y position can be negative.
I personally can´t use this workaround because android has no bounce effect and therefore the y pos is never negative.

@hartez hartez changed the title IOS: Scrollview inside Refrehview IOS: Scrollview inside Refreshview Dec 20, 2022
@rmarinho rmarinho self-assigned this Mar 31, 2023
@rmarinho
Copy link
Member

This seems to be fixed by #11624
I tested on the release branch it seems fixed so it will be on the next 7.0 service release 5.

@ghost ghost locked as resolved and limited conversation to collaborators Apr 30, 2023
@samhouts samhouts added fixed-in-8.0.0-preview.1.7762 Look for this fix in 8.0.0-preview.1.7762! fixed-in-7.0.52 Look for this fix in 7.0.52 SR1.1! labels May 10, 2023
@samhouts samhouts modified the milestones: Backlog, .NET 7 + Servicing May 22, 2023
@Eilon Eilon removed legacy-area-controls Label, Button, CheckBox, Slider, Stepper, Switch, Picker, Entry, Editor area-layout StackLayout, GridLayout, ContentView, AbsoluteLayout, FlexLayout, ContentPresenter labels May 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-controls-refreshview RefreshView area-controls-scrollview ScrollView fixed-in-7.0.52 Look for this fix in 7.0.52 SR1.1! fixed-in-7.0.100 fixed-in-7.0.101 fixed-in-8.0.0-preview.1.7762 Look for this fix in 8.0.0-preview.1.7762! platform/iOS 🍎 t/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants