Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scroll star/read buttons #115

Closed
1 task done
rugk opened this issue Mar 16, 2017 · 16 comments
Closed
1 task done

Scroll star/read buttons #115

rugk opened this issue Mar 16, 2017 · 16 comments

Comments

@rugk
Copy link

rugk commented Mar 16, 2017

Usually you read an article first before staring it or clicking on "is read". However, when you are at the end of the article (when the whole one is downloaded), you'd have to scroll up just to do.
I'd like it if the buttons were reachable directly, so e.g. just make them fixed and scroll them with the browser window.

@BernhardPosselt
Copy link
Member

That's what keyboard shortcuts are for (simply press s) ;)

@rugk
Copy link
Author

rugk commented Mar 16, 2017

That requires you know them… 😃 Not all people do. (Or some just don't like to use their keyboard.)

@BernhardPosselt
Copy link
Member

Use the scroll wheel then :D

@BernhardPosselt
Copy link
Member

To elaborate: sure, using the mouse often makes you cover greater distances. This is true for nearly every application, that's why people invented shortcuts. You shouldn't have to duplicate buttons in the UI just because certain people refuse to use a keyboard for efficiency.

I was an avid mouse user myself but I pretty much only use keyboard shortcuts these days. Just give it a try ;)

@rugk
Copy link
Author

rugk commented Mar 17, 2017

Yes all right, I could do, but this issue is not about me. It's for all users of this app and you can't tell them all "Hey use this fast shortcut to star your text.". That's why I requested a solution, which is easy to get by all users and does not require any explanation, so finally it improves the UX IMHO.

And I did not suggest duplicating buttons, that's indeed maybe not so nice. Just let them in the view when the user scrolls, i.e. make them fixed.

@BernhardPosselt
Copy link
Member

Just let them in the view when the user scrolls, i.e. make them fixed.

That's a good point indeed. Would need to test something first in order to get a feel for it. Usually fixed headers are not that nice to use since they hide content

@rugk
Copy link
Author

rugk commented Mar 23, 2017

hide content

If there is space show them at the right of the news text. I've always have some empty space there…

@nasredin
Copy link

first i have to say, good job :) but i wish some buttons(at least "next article") also, its poor experience to scroll on tablets:( i tried cloudnews app for ios but its also bad experience and very buggy.

@BernhardPosselt
Copy link
Member

BernhardPosselt commented Mar 27, 2017

@nasredin at least on the android app the experience is very neat. As for mobile devices: use the mobile apps :)

Very likely buttons for navigating between articles won't be added. Scrolling should be the default and it should be a good experience. In general the web experience on lower end mobile devices and on iOS is known to "not rock", mainly because Safari is a piece of trash and Apple does not allow you to install a different browser engine.

All I can recommend you in that case is to use CloudNews and file your issues there or get an Android device.

@nasredin
Copy link

so first what i want to say is, i have no problem with scrolling in another sites including news apps, i have tested. But if u are serious with your reply i think, we both don't have to say each other... its like when u say, u should move to another apartment if u want use my curtains.... nonsense... but thx for your reply

@BernhardPosselt
Copy link
Member

@nasredin don't take it personal, this is a typical text communication issue where intent is lacking because you can't hear/see one another. Nothing is meant in a mean fashion ;)

@nasredin
Copy link

yes its ok, i moved to tt-rss and made shortcut in nextcloud, so far, maybe u will change decision later :) i will keep my eyes on your project.

@Grotax
Copy link
Member

Grotax commented Mar 1, 2019

Can this be closed by #338 ?

@rugk
Copy link
Author

rugk commented Mar 1, 2019

@skjnldsv said it closes it, so I guess it's okay to close it.

BTW, @skjnldsv, you can (automatically) let issues close when a PR is merged with a special keyword.

@skjnldsv
Copy link
Member

skjnldsv commented Mar 1, 2019

@rugk oh I have no idea if that closes it or not.
I just merged it because I've got the permissions, but I do not maintain this app. :)

@Grotax Grotax closed this as completed Mar 1, 2019
@rugk
Copy link
Author

rugk commented Mar 1, 2019

sorry, meant @oherst. I mentioned the wrong person.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants