This repository was archived by the owner on Feb 20, 2023. It is now read-only.
This repository was archived by the owner on Feb 20, 2023. It is now read-only.
[Pull To Refresh][Bug] Avoid accidental refreshes by forcing a cooldown period after the last scroll gesture #16661
Closed
Description
Steps to reproduce
- On a scrollable page, scroll down.
- Scroll up to the top with a series of quick pans / flings.
Expected behavior
No pull refresh should be triggered during gestures that were intended to only scroll.
Actual behavior
As soon as the top edge of the page is hit, the next gesture instantly triggers pull to refresh.
This does not happen in Chrome. Chrome seems to enforce a cooldown period before pull to refresh is triggered. See this video for a comparison: quick-consecutive-flings.mp4
From experimenting, it feels like the cooldown period is somewhere between 300 and 500ms.
Device information
- Android device: Moto G5
- Fenix version: Firefox Nightly
┆Issue is synchronized with this Jira Task
Metadata
Metadata
Assignees
Type
Projects
Milestone
Relationships
Development
No branches or pull requests
Participants
Activity
Mugurell commentedon Feb 2, 2021
Judging from the video and a quick test in Chrome,
There is no cooldown per se.
What happens on Chrome is that a swipe up and then down will trigger the overscroll effect instead of the pull to refresh throbber like it happens on Fenix.
Then if you continue to swipe down while the edge effect hasn't finished the gesture will trigger again the overscroll effect and it would be possible to keep this behavior with such swipes for a loong time.
That being said, we should probably do the same. Thank you for filing this!
stale commentedon Aug 1, 2021
See: #17373 This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
ghost commentedon Sep 3, 2021
I have a similar issue on div with scrollable layout (site without mobile layout), everytime I try to scroll down the content of div I got a refresh and not a scroll.
I use the tricks scroll down and to right. but not always works.
stale commentedon Mar 2, 2022
See: #17373 This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
stale commentedon Nov 23, 2022
See: #17373 This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
csadilek commentedon Dec 23, 2022
Moved to bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1807072
Change performed by the Move to Bugzilla add-on.