Problem/Motivation

In the case of secondary menus or vertical side menus, the nav component should allow dropdowns to be optional.

CommentFileSizeAuthor
#4 radix-nav-dropdown4.png19.92 KBchrisck
#4 radix-nav-dropdown3.png12.5 KBchrisck
#4 radix-nav-dropdown2.png14.83 KBchrisck
#4 radix-nav-dropdown1.png16.95 KBchrisck

Issue fork radix-3564224

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

chrisck created an issue. See original summary.

chrisck’s picture

chrisck
he/him
Location BC, Canada
commented
Related issues: +#3260136: Make nav default to dropdown links optional

chrisck opened merge request !155

chrisck’s picture

chrisck
he/him
Location BC, Canada
commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new radix-nav-dropdown1.png16.95 KB
new radix-nav-dropdown2.png14.83 KB
new radix-nav-dropdown3.png12.5 KB
new radix-nav-dropdown4.png19.92 KB

Here are a few screenshots of the MR in current state. Setting to Needs review.

When nav component is originally configured where is_dropdown is not specified or true (default):
radix nav dropdown

When nav component is configured where is_dropdown is false:
radix nav dropdown

When nav component is configured where is_dropdown is false and all parent menu items are expanded:
radix nav dropdown

When nav component is configured where is_dropdown is false and alignment is vertical:
radix nav dropdown

doxigo made their first commit to this issue’s fork.

doxigo’s picture

doxigo commented
Status: Needs review » Fixed

Hey Chris, thanks for the MR, there were some issues with the current MR and wasn't rather complete and I just wrapped it up locally. Fixed and closed your MR, but credited.
Thanks

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.

  • doxigo committed 05ab91bd on 6.0.x 
    Issue #3564224: Add optional dropdown to nav and fix dropdown_direction...

doxigo closed merge request !155

Status: Fixed » Closed (fixed)

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