-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Too many menus on settings page #2271
Comments
OpenID/External seems like we could merge them. 2FA should go under Pasword. Avatar under Profile. |
We could also merge SSH / GPG Keys and Applications into "Keys / Tokens" |
Merge Profile and Avatar into Profile |
should be a part of #1346 |
Unfortunately I was to late to review #2695 |
@daviian Delete account could be moved into |
@lafriks Profile is more like the publicly visible part of your account. Whereas |
Rename |
@lunny I think this would be great to put under my proposed "Account" tab |
@daviian Yes, your idea is better. |
I think left sidebar is better as original. top menu is tooo crowded |
@renothing left menu does not work well on narrow screens. Also, as this issue says, there are too many menus (many can be combined). |
flexible.js maybe usefull |
I'll give it a shot and prepare an alternative menu / subpage structure for the settings page before actually implementing it. |
So I came up with following structure:
First level represents the menu items. Second level refers to items that are moved onto that page. |
Any objection against above proposed menu readjustments? |
LGTM :) |
@daviian Why have SSH / GPG separate from Security settings? |
@Guard13007 At first I thought I move SSH / GPG Keys to Security as well, but hesitated because of the GPG Keys part as they don't have something todo with Security/Authentication. |
I don't think current layout good enough still. Do not change for change, submenu need two steps for users, and horizontal layout crowded. if next a new option added, so will you add new subgroup again ??why don't change it back to sidebar menu like github or gitlab? it's simple and rational more than current. |
Oh I don't change for a change, I really think that some menu items should be bundled together no matter if the submenu is horizontal or vertical. |
@renothing Having a vertical menu is a big disadvantage for those who need to work with smaller screens. |
@Guard13007 I don't think these users are most common, will you use a mobile phone to work with github in most case? horizontal menu waste too much layout spaces |
So screw the minority because you don't like the look of something? The horizontal layout isn't that overcrowded, and the suggested re-arrangement would solve that problem easily without having to resort to sacrificing usability on smaller screens. Also, I said nothing about mobile phones, I said smaller screens. That said, I do often need to check things on my phone as well, so yes, on a mobile phone is useful too. |
@lunny I think we can close this issue now. |
The text was updated successfully, but these errors were encountered: