You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When refreshing Dashboard, the refresh animation that shows up is an incomplete circle animation instead of the traditional circling cycle animation on every other apps. (Don't judge me for phrasing it this way.)
What ReVanced Manager does:
screen-20231016-004038.2.mp4
What it should be doing:
screen-20231016-004355.3.mp4
Also, the refresh icon starts showing up all the way from the notification shade instead of showing up within the manager. It should start from the point in the Dashboard where the padding is when scrolling down.
Without scrolling:
When scrolled down a bit:
The refresh icon should start after the padded area of Dashboard, like it's coming from underneath the padded area.
Motivation
I checked out the examples provided in the repository and MaterialClassicHeader seems to be the one which was implemented in ReVanced Manager (or looks like what was intended). Correct me if I'm wrong.
P.S: Attaching the gif for further clarity
Additional context
No response
Acknowledgements
This request is not a duplicate of an existing issue.
I have chosen an appropriate title.
All requested information has been provided properly.
The issue is solely related to the ReVanced Manager
The text was updated successfully, but these errors were encountered:
ILoveOpenSourceApplications
changed the title
feat(Refreshing Animation): Improve the refreshing animation
feat(Refresh Animation): Improve the refreshing animation
Jan 15, 2024
Yup, I'd like to try this one. Let me handle this issue.
TL;DR
The issue seems to be with the current layout, where the refresh indicator wraps the AppBar and Content, causing the subsequent overscroll to show the indicator over the scrollable app bar and content.
Long Version
I think the ReVanced dashboard is using the [RefreshIndicator] widget which needs to wrap a scrollable, that when overscrolls shows a refresh jndicator, in this case it is a [CustomScroll] child having the [SliverAppbar] and then the [SliverChildList], which is the reason for the "issue."
Because the RefreshIndicator by default wraps the custom scroll in its extent, and has an edge property of 0 (which starts showing the indicator in that position), the indicator starts at the start, on the topmost position of the custom scroll (having the appbar and content )
It should be relatively easy to fix; it just needs a gap on the edge property (from which starts showing the refresh indicator) or by changing the layout.
Feature description
When refreshing
Dashboard
, the refresh animation that shows up is an incomplete circle animation instead of the traditional circling cycle animation on every other apps. (Don't judge me for phrasing it this way.)What ReVanced Manager does:
screen-20231016-004038.2.mp4
What it should be doing:
screen-20231016-004355.3.mp4
Also, the refresh icon starts showing up all the way from the notification shade instead of showing up within the manager. It should start from the point in the
Dashboard
where the padding is when scrolling down.Without scrolling:
When scrolled down a bit:
The refresh icon should start after the padded area of
Dashboard
, like it's coming from underneath the padded area.Motivation
I checked out the examples provided in the repository and
MaterialClassicHeader
seems to be the one which was implemented in ReVanced Manager (or looks like what was intended). Correct me if I'm wrong.P.S: Attaching the gif for further clarity
Additional context
No response
Acknowledgements
The text was updated successfully, but these errors were encountered: