search

Huessenbergnetz / Fuoten

ownCloud / Nextcloud News App client for Sailfish OS
GNU General Public License v3.0
8 stars 5 forks source link

removes the pushup menu from the article view #41

Closed blizzz closed 5 years ago

blizzz commented 5 years ago

@buschmann23 first, are you still maintaining the app? There was not much movement lately, sadly.

This PR essentially removes the PushUp menu from the article view (on the phone). That's mostly a UX improvement, a least from my PoV: while reading an article I am constantly scrolling down, the eyes on one position, so I don't know or see when the end of the articles comes. But then, the menu becomes active and too often I end up on "Scroll on top", loosing my reading position.

Functionality is not really lost: auto-scrolling to top works with the native Silica scroll decorations, the other items are available on the top menu.

It is understandable if you would not like that approach, but then we can look for a way to make it work for everybody? Settings are always possible, but ideally they are avoided as much as possible… All provided you are still interested in maintaining the app, of course.

buschmann23 commented 5 years ago

I am still working on my apps, at least as I have enough time for it. :) Thank you for your proposal. But as a KDE user I like settings. ;) So I will add a setting to enable the push up menu - with a default to enabled.