Reorganize Navigation menu items and make them consistent with other similar pages in core

Created on 27 March 2025, 6 days ago

Problem/Motivation

Navigation module menu items for its settings and block layout are defined with different weights, hence they do not appear together in a clean core installation. That could be confusing for site administrators.

Proposed resolution

Define the same weight for both menu items. Leaving the value empty to use the default value might be the preferred approach
While we are here, rename Navigation Settings route controller and menu item to Navigation settings to be consistent with other settings pages in core
While we are here, add a simple description to Navigation settings menu item to be consistent with other core settings menu links. A simple string like Manage navigation settings. would be enough.

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

πŸ“Œ Task
Status

Active

Version

11.0 πŸ”₯

Component

navigation.module

Created by

πŸ‡ͺπŸ‡ΈSpain plopesc Valladolid

Live updates comments and jobs are added and updated live.
  • Novice

    It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @plopesc
  • πŸ‡ΊπŸ‡ΈUnited States ivagold Philadelphia, PA

    I'm at DrupalCon Atlanta and am interested in giving it a try! Let's see how far I get. ;-)

  • Pipeline finished with Failed
    6 days ago
    Total: 392s
    #458984
  • Pipeline finished with Success
    6 days ago
    Total: 452s
    #458991
  • πŸ‡ΊπŸ‡ΈUnited States ivagold Philadelphia, PA

    Ok, It's updated, and when I test it, the navigation menu items are reordered and match in text case:

  • πŸ‡ͺπŸ‡ΈSpain plopesc Valladolid

    Good progress so far! :)

    Added a minor comment to the MR to reduce a bit the file footprint. Besides that, would be great if the Navigation settings form page could be updated as well. It is defined in navigation.routing.yml file.

    Thank you!

  • Pipeline finished with Success
    6 days ago
    Total: 347s
    #459030
  • πŸ‡ΊπŸ‡ΈUnited States ivagold Philadelphia, PA

    I've changed the case in that page's title to match the link in the menu. Is that the only thing that needed updating?

  • πŸ‡ΊπŸ‡ΈUnited States ivagold Philadelphia, PA
  • Pipeline finished with Success
    6 days ago
    Total: 2006s
    #459049
  • πŸ‡ͺπŸ‡ΈSpain plopesc Valladolid

    It looks great!

    Marking as RTBC. Thank you!

Production build 0.71.5 2024