search

msasikanth / twine

Twine: A multiplatform RSS reader built using Kotlin and Compose
Apache License 2.0
1.77k stars 73 forks source link

Move scroll to top to bottom of the screen #720

Closed FluffyDiscord closed 3 months ago

FluffyDiscord commented 3 months ago

Describe the bug Dismissing bottom bar doesnt update the scroll to top button position.

To Reproduce Steps to reproduce the behavior:

  1. Open the app
  2. Dismiss bottom bar
  3. Scronn down until button appears

Screenshots Screenshot_20240821_090029_Twine

Feed Information

Device Information

Stacktrace

FluffyDiscord commented 3 months ago

On the side note - keep the bottom bar hidden, if I dissmiss it, it takes too much space for no good reason. It vould be an option in settings or in the top right menu button dropwdown

msasikanth commented 3 months ago

Bottom bar is the navigation area in this app for feed management. It won't be hidden.

Currently there is a bug which is causing it to be hidden when you swipe down. But that will be fixed in later releases.

With bottom bar in place, the scroll to top button is exactly where it's supposed to be.

FluffyDiscord commented 3 months ago

Oh, i found that actually useful, because as i mentioned, it steals pretty big chunk of the reader area

msasikanth commented 3 months ago

Oh, i found that actually useful, because as i mentioned, it steals pretty big chunk of the reader area

I hear you. I think one of the options we can try out is hiding app bar which also takes up space during scroll. That way users have more space to explore the articles.