Remove redundant secondary menu links

Created on 19 June 2025, 16 days ago

Problem/Motivation

Several of the menu links in the new documentation pages have only one sub-item, so the drop-downs do not add any value, and in fact it just increases the number of clicks to get to a page.

Proposed changes

If there is only one page within a top-level menu item, have that page as the menu link and remove the drop-down to second level.

📌 Task
Status

Active

Component

Documentation

Created by

🇬🇧United Kingdom jonathan1055

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @jonathan1055
  • First commit to issue fork.
  • Merge request !7nav and site name updates → (Open) created by quietone
  • 🇳🇿New Zealand quietone

    I added this back in but the changes here are deployed to the 'real' URL. Did I make a mistake?

    # Temporary change to rebuild the docs in MR without a commit to default branch.
    pages:
      rules:
        - !reference [ .skip-pages-rule ]
        - exists:
            - mkdocs.yml
          when: on_success

    I also noticed a bunch of information errors at https://git.drupalcode.org/project/coding_standards/-/jobs/5613053#L144. So there is more cleanup to do. How about expanding the scope of this to include that?

  • 🇬🇧United Kingdom jonathan1055

    I added this back in but the changes here are deployed to the 'real' URL. Did I make a mistake?

    That is the expected behavior when those lines are added in a MR and the pipeine is run by a maintainer. Unfortunately we don't have any solution for this yet, and its not satisfactory at all. If I were to submit a pipeline on this MR the pages would be built to a different url, and the live site would not be affected until the MR was committed. This is the same as what I noted in the issue summary on 📌 Formatting fixes Active . I have updated Gitlab Templates issue 📌 Only rebuild documentation pages when source has changed Active so hopefully we can make some progress there.

    I also noticed a bunch of information errors

    Well spotted. I wonder if those could be highlighted better in the log, or somehow we draw attention to them. Yes, definitely fix in this MR (I think you have done some already)

  • 🇬🇧United Kingdom jonathan1055

    Just checked, and confirm that all of the redundant links have been removed, and also you have fixed all of the info/warnings about missing urls. So this is RTBC from me.

    • quietone committed e45d2cde on main
      Issue #3530937 by quietone, jonathan1055: Remove redundant secondary...
  • 🇳🇿New Zealand quietone

    @jonathan1055, thanks.

Production build 0.71.5 2024