Problem/Motivation

Add menu support for sites using new Drupal Navigation module in core

Proposed resolution

  • Upgrade templates to support the new Navigation module shipping with Drupal 10.3+
  • With these changes continue to support historical menu structure

Issue fork prototype-3578032

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

jldust created an issue. See original summary.

jldust’s picture

jldust commented
Assigned: Unassigned » philip_stier
jnettik’s picture

jnettik
Primary language English
Location Denver, CO
commented
Status: Active » Postponed (maintainer needs more info)

I've been trying to recreate the issue that started this and I can't now. I reverted code I wrote to "fix" things on another project and it's working fine now. Updating this to postponed until we can confirm there is an issue to fix.

jldust’s picture

jldust commented
Assigned: philip_stier » jldust
Status: Postponed (maintainer needs more info) » Needs review

I was able to test & confirm that the primary issues previously addressed by enabling the Notification module on a fresh install no longer seems to break the admin interface. That said I did push a small attribute validation to resolve a menu error I was seeing during testing.

jldust opened merge request !225

jldust’s picture

jldust commented
Status: Needs review » Reviewed & tested by the community

This has been merged in and will be included in the next tagged release.

  • jldust committed 21ebeb84 on 5.x 
    feat: #3578032 Add support for Navigation module

By: jldust
jldust’s picture

jldust commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

jldust’s picture

jldust commented
Status: Fixed » Closed (fixed)