FreshRSS / FreshRSS

A free, self-hostable news aggregator…
https://freshrss.org
GNU Affero General Public License v3.0
9.56k stars 818 forks source link

[Feature] Move logout button to the end of list #5716

Open ketozhang opened 11 months ago

ketozhang commented 11 months ago

Is your feature request related to a problem? Please describe. I’m always frustrated1 when I click the gear icon and accidentally log out. I also accidentally clicked logout from the settings page.

[1]: I love how the issue template started my sentence

Describe the solution you’d like Please move logout to the last item in settings. It is common practice, bet its part of some UX consensus---for example, github: image

Frenzie commented 11 months ago

I'm inclined to agree. 👍

vdbhb59 commented 11 months ago

Agreed. I too get logged out due to page getting aligned upon full load. Very frustrating.

math-GH commented 11 months ago

Thanks for giving feedback so that we could improve the UX.

Before we discuss a solution let me understand the issue better.

Which theme do you use? Which FreshRSS version? Mobile or Desktop view? What exactly do you do? Could you please describe step by step your actions or could you please record it as video/GIF?

vdbhb59 commented 11 months ago

Thanks for giving feedback so that we could improve the UX.

Before we discuss a solution let me understand the issue better.

Which theme do you use? Which FreshRSS version? Mobile or Desktop view? What exactly do you do? Could you please describe step by step your actions or could you please record it as video/GIF?

Responses below: Irrespective of the theme the logout button is at 2nd position 1.21.0 Mobile/Desktop - both For me happens mostly when loading a page and clicking on it in error upon trying to click on first instance It also happens when in error or by mistake the logout button gets clicked while trying to click some other button

ketozhang commented 11 months ago

Ditto to @vdbhb59 with the difference that I am use https://feeds.flossboxin.org.in (I am a fresh user just trying this out before self-hosting).

vdbhb59 commented 11 months ago

Ditto to @vdbhb59 with the difference that I am use https://feeds.flossboxin.org.in (I am a fresh user just trying this out before self-hosting).

That is my instance only (I run it). So, similar issue. Though, I found it same on my personal host too on a different server.

Frenzie commented 11 months ago

I've never accidentally logged out but I can see how it's dangerously close to other things. But also, the position of close/logout everywhere else conditions us to look at the bottom.

HarikaB11 commented 10 months ago

Hi @ketozhang , I would like to work on this issue.

Alkarex commented 10 months ago

Go ahead @ketozhang 👍🏻

MananDhiman commented 10 months ago

Please take a look at my commit here and if any changes let me know

If everything is okay I'll create a pull request

It's my first pull request so I appreciate all help

Alkarex commented 10 months ago

@MananDhiman Just make a PR. It makes it easier to provide feedback, further edits, and tests