clragon / e1547

A sophisticated e621 browser
https://e1547.clynamic.net/
GNU General Public License v3.0
209 stars 16 forks source link

ScrollToTop Button replacement #54

Closed clragon closed 2 years ago

clragon commented 2 years ago

The scroll to top button adds movement to the screen and does not feel like a natural part of the app.

A solution would be to retry adding the double tap functionality to the appbar.

clragon commented 2 years ago

By creating a central default app bar that makes use of the defaut scroll controller, we have achieved the desired results.

Of note is that the default appbar prohibits tapping the appbar title by wrapping it in an ignorepointer. The reason for this is because the title will otherwise prevent the user from tapping the appbar flexible space directly and triggering a scroll to top.