Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix hidden active elements in admin menu #3926

Merged
merged 1 commit into from
Feb 21, 2020
Merged

Fix hidden active elements in admin menu #3926

merged 1 commit into from
Feb 21, 2020

Conversation

javierm
Copy link
Member

@javierm javierm commented Feb 20, 2020

Background

In the admin menu, some links take you to a section, and some links open a submenu with more links.

When we disable the "multi-open" property of the menu and the active element is a link which takes you to a section, Foundation will hide it whenever we click a link which opens a submenu.

Objectives

Make sure active menu items aren't hidden when navigating through the menus.

Visual Changes

Before these changes:

The current element is hidden when opening a submenu

After these changes:

The current element is present when opening a submenu

In the admin menu, some links take you to a section, and some links open
a submenu with more links.

When we disable the "multi-open" property of the menu and the active
element is a link which takes you to a section, Foundation will hide it
whenever we click a link which opens a submenu.

The easiest solution is to enable "multi-open" property.
@javierm javierm added the Bug label Feb 20, 2020
@javierm javierm self-assigned this Feb 20, 2020
@javierm javierm added this to Reviewing in Roadmap via automation Feb 20, 2020
Roadmap automation moved this from Reviewing to Testing Feb 20, 2020
@javierm javierm merged commit 5f17718 into master Feb 21, 2020
Roadmap automation moved this from Testing to Release 1.1.0 Feb 21, 2020
@javierm javierm deleted the admin_menu branch February 21, 2020 12:02
@PierreMesure
Copy link
Collaborator

Well done! Pretty sure I opened an issue about it two years ago but can’t find it.

smarques pushed a commit to venetochevogliamo/consul that referenced this pull request Apr 29, 2020
Fix hidden active elements in admin menu
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Roadmap
  
Release 1.1.0
Development

Successfully merging this pull request may close these issues.

None yet

4 participants