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

Move all settings menu items under Configuration > Settings menu #11942

Open
MikeAlhayek opened this issue Jul 3, 2022 · 22 comments · May be fixed by #17112
Open

Move all settings menu items under Configuration > Settings menu #11942

MikeAlhayek opened this issue Jul 3, 2022 · 22 comments · May be fixed by #17112
Milestone

Comments

@MikeAlhayek
Copy link
Member

MikeAlhayek commented Jul 3, 2022

Is your feature request related to a problem? Please describe.

Currently, site settings provided by modules are scattered on the menu in many places. Form user experience, it would be nice to place all settings under one dedicated location on the menu for easy of access and quickly know where are all the settings located.

Also, the text style on the menu items isn’t unified.

Describe the solution you'd like

Move any site setting or configurations under the “Configuration > Settings” menu

Additionally, we may want to unify the naming by using a common style. Either a title case or sentence case for every menu item. I think using sentence style will look good on the menu item as per the recommendation here

Here is how the new menu structure may look like

Configuration > Settings > Search
Configuration > Settings > Security > User login
Configuration > Settings > Security > User change email
Configuration > Settings > Security > User registration
Configuration > Settings > Security > User reset password
Configuration > Settings > Security > Security header
Configuration > Settings > Security > HTTPS
Configuration > Settings > Security > CORS
Configuration > Settings > Security > reCaptcha
Configuration > Settings > Security > OpenID Connect > Authentication client
Configuration > Settings > Security > OpenID Connect > Authorization server
Configuration > Settings > Security > OpenID Connect > Token validation
Configuration > Settings > Security > OpenID Connect > Management
Configuration > Settings > Security > Authentication > Facebook
Configuration > Settings > Security > Authentication > Twitter
Configuration > Settings > Security > Authentication > GitHub
Configuration > Settings > Security > Authentication > Google
Configuration > Settings > Security > Authentication > Microsoft
Configuration > Settings > Security > Authentication > Azure Active Directory

@MikeAlhayek MikeAlhayek changed the title Move all setting menu items under configuration > Settings menu Move all settings menu items under Configuration > Settings menu Jul 4, 2022
@sebastienros
Copy link
Member

Needs some discussion, the current menu organization was discussed and designed this way.

@MikeAlhayek
Copy link
Member Author

@sebastienros sounds good. I just think centralizing settings related items into one place will make the UI easier to navigate. For anyone looking for settings, they are always in the same place.

@Skrypt
Copy link
Contributor

Skrypt commented Jul 9, 2022

I kind of agree with him that "Security --> User / Roles" should be separated from the "Configuration --> Security" settings. Makes sense to me to be able to quickly find an admin UI to manage users/roles but having all the settings in there is not necessary. We don't need to access these settings most of the time.

@ns8482e
Copy link
Contributor

ns8482e commented Jul 15, 2022

IMHO - end user looking Configuration menu they tend to think configuring site in terms of general setting related to the site
and Setting of each vertical functional group to have its own settings i.e. what we have today Security -> Settings

Also having more than 2 level of menus are annoying and not good UI experience

@hishamco
Copy link
Member

Also having more than 2 level of menus are annoying and not good UI experience

Could you please add anything related to UI/UX to the related issue to mak it easier for tracking

@Skrypt
Copy link
Contributor

Skrypt commented Jul 15, 2022

@ns8482e I agree with you also. Maybe that's the compromise that has been done so that we don't have too many levels of sub-menus. Either way, I don't think it is currently causing any issue as everything is accessible.

@MikeAlhayek
Copy link
Member Author

MikeAlhayek commented Jul 16, 2022

@Skrypt @ns8482e may be we can create a new menu in the root called "Settings" which will be under "Configurations" or drop "Settings" level altogether and list all settings directly under "Configurations"

I think it is a MUCH better user experience to group all settings/configurations under the same node. IMO, placing all settings in one place removes the guessing from the user experience. Settings are not always accessed and should be a second class citizens but should be grouped.

@sebastienros
Copy link
Member

Let's find the meeting where we discussed about Configuration vs Setting menu items. And when these items were first introduced.

@sebastienros sebastienros added this to the 1.x milestone Aug 4, 2022
@MikeAlhayek
Copy link
Member Author

@sebastienros were you able to find the meeting?

@MikeAlhayek
Copy link
Member Author

@sebastienros @Piedone I think we should revisit this old request. I think it is still a valid issue. We should move all the settings into one central place instead of having it all over the place.

@Piedone
Copy link
Member

Piedone commented Oct 14, 2024

We can talk about this at tomorrow's meeting.

While I'm in favor of cleaning up the admin menu, only keeping in mind the following:

  • We should have as few levels as possible while balancing it with having as few menu items on each level as possible.
  • We should avoid using both "configuration" and "settings" because these are synonyms.
  • We should go outside of the circle of a few core maintainers and see what users say. Perhaps also ask some UX person and see what other systems do.
  • Whatever we do, we should only do it for v3. Changing menu item labels necessitates localization, which while not technically breaking, still something outside of "I update and everything just keeps working". It's also annoying to learn a new menu structure. And if you're a module author, you may notice that suddenly your module's menu items are in a very different context.

All that being said, I'm for having topic-oriented top-level menu items (like Content, Search, Security) in contrast to moving all configuration-related items under a separate menu. WordPress seems to be doing the "all settings under one menu" approach, Umbraco more of a mixed one. BTW I like the Tools menu from WP:

{27B2FD38-D618-4F14-857A-92CC75616B86}

A lot of stuff we have under Configuration currently is really tooling.

@MikeAlhayek
Copy link
Member Author

@Piedone I agree this would have to be in a major release as people will need to adjust their menu accordingly. I also agree that we should reduce the levels of the menu.

I think Design should be replaced with Appearance. And "Tools" would be a better option that "Configuration"

We'll talk more about this tomorrow and if we all agree, we'll tag this issue with 3.0 and then put the agreed menu structure here so it can be done when working on 3.0

@MikeAlhayek
Copy link
Member Author

MikeAlhayek commented Oct 22, 2024

Here is a prototype of moving the settings into their own menu away from the main menu. With this prototype, we achive

  • Less menu items being adding on every page admin request
  • Less menu levels
  • More focused menu on the site navigation while having a settings focused menu.
  • We can add a settings icon on the top navbar that would take the user to the site settings if we don't want to have it on the settings.
  • Adding Tools menu may be a better term to use than "Configuration".

image

image

image

image

This prototype can be found on this branch https://github.com/OrchardCMS/OrchardCore/tree/ma/rearrange-admin-menu

@Piedone
Copy link
Member

Piedone commented Oct 23, 2024

To add here too what I told on the meeting yesterday: I'm against having tertiary navigation (besides the main menu on the left and the editor tabs). I'd start this issue by having a separate Tools and Configuration (or Settings) menu and move all the non-config features from the current Configuration menu (Import/Export, Recipes, GraphiQL, perhaps also Tasks) to Tools.

@MikeAlhayek
Copy link
Member Author

Here some of the apps that have a dedicated menu for settings

Microsoft Outlook
image

Shopify

image

Discord

image

@MikeAlhayek
Copy link
Member Author

Updated screenshots with settings search and moving the settings to the top level of the menu.

image

Search
image

image

image

image

@gvkries
Copy link
Contributor

gvkries commented Oct 24, 2024

I would absolutely appreciate having all settings in a single spot. That is the only thing I find necessary, the other improvements like a "Tools" menu are nice to have IMHO. Regarding the new menu on the right, I understand Mikes reasoning, but I don't think it is required.

But can't we allow editing existing menu items with the admin menu feature? This way, everyone can create their own preferred structure ;-)

@MikeAlhayek
Copy link
Member Author

Regarding the new menu on the right, I understand Mikes reasoning, but I don't think it is required.

You are right, it's not required. We can just put all the settings into one tab and keep the main menu busy. But, do you see a better user experience by create one site navigation focus menu and another settings focus menu?

@gvkries
Copy link
Contributor

gvkries commented Oct 24, 2024

I'm not an UX expert, but personally I can see advantages of a separate settings menu on the left. This doesn't really introduce a new UI concept, but makes the main menu smaller and more focused for tasks used more often. But again, I don't think this is something that has such a big impact and given the push back from others, I don't think we should continue pursuing it.

@Piedone
Copy link
Member

Piedone commented Oct 24, 2024

Another aspect to consider is the collapsed menu, which is especially important on smaller screens:

{55AE74F6-385D-4724-B397-F375F5EFE5E1}

@domonkosgabor
Copy link
Contributor

Discussion about this issue from this week's podcast: https://youtu.be/i1dEWzKl2pw?si=uHucNH1_4XuvThrE&t=1586

@MikeAlhayek
Copy link
Member Author

Another aspect to consider is the collapsed menu

Another reason why we should try to keep the left menu focused and have less menu items (all settings). Less items in the main menu will make navigation much simpler of small devices and large once too.

@MikeAlhayek MikeAlhayek linked a pull request Dec 4, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
8 participants