Problem/Motivation

When enabling the setting `show_groups_in_menu`, in combination with admin_toolbar, the `Admin > Content > Site Settings` link gets children for each configured site settings group.

When using the Navigation menu instead, a user needs to select "Content" in the navigation, and then use the "Site settings" tab on that page.

We would like to more easily reach the site settings, including group child links, if enabled.

Moreover, when the `show_groups_in_menu` setting is enabled, the group menu items are added randomly in the main navigation tree (between "Structure", "Appearance", etc.. links), which looks clunky.

Steps to reproduce

- Install navigation core module
- No site settings are available.

Proposed resolution

The Navigation module provides a "Content" menu, where it places the default content links ("Create", "Content", "Media", ...
When this menu exists, add the "Site settings" + optional group children to it.

Remaining tasks

Create MR

User interface changes

Before:

Before

After

After

CommentFileSizeAuthor
#4 3507687--site-settings-navigation-content--4.patch2.9 KBsvendecabooter
Screenshot from 2025-02-19 11-07-31.png31.08 KBsvendecabooter
Screenshot from 2025-02-19 11-06-20.png21.18 KBsvendecabooter

Issue fork site_settings-3507687

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

svendecabooter created an issue. See original summary.

svendecabooter opened merge request !52

svendecabooter’s picture

svendecabooter
he/him
Primary language Dutch
Location Gent
commented
Status: Active » Needs review
svendecabooter’s picture

svendecabooter
he/him
Primary language Dutch
Location Gent
commented
StatusFileSize
new 3507687--site-settings-navigation-content--4.patch2.9 KB

Adding patch file as well for composer patch workflows.

svendecabooter’s picture

svendecabooter
he/him
Primary language Dutch
Location Gent
commented

Ideally this would get a dedicated icon too, but I'm no designer / frontend dev at all :)

scott_euser’s picture

scott_euser commented
Status: Needs review » Needs work

Thanks!

  1. Maybe we can use the same icon as Configuration for now?
  2. Has a merge conflict with your other issue that I just merged changing menu structure
svendecabooter’s picture

svendecabooter
he/him
Primary language Dutch
Location Gent
commented
Status: Needs work » Needs review

Merge conflict fixed, and configuration icon added.

scott_euser’s picture

scott_euser commented
Status: Needs review » Fixed

Excellent, works great thank you!

scott_euser’s picture

scott_euser commented
Status: Needs review » Fixed
svendecabooter’s picture

svendecabooter
he/him
Primary language Dutch
Location Gent
commented

Thanks for committing!
Could you perhaps tag a new release, now that a bunch of issues have been marked fixed?

scott_euser’s picture

scott_euser commented

Done sorry just didn't want to make a release until I was back from holiday - and didn't want to publicly announce I was on holiday until getting back :)

Thanks for the contributions!

svendecabooter’s picture

svendecabooter
he/him
Primary language Dutch
Location Gent
commented

Thanks, no worries. Hope you enjoyed your holidays :)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.