Motivation

A use case for this module can be: display the group menu and use another general menu as fallback.

Proposed resolution

To achieve that it would be handy to have a condition plugin to control visibility. Place the group menu and the fallback drupal menu in the same region next to each other. Configure a condition for the fallback menu to be visible only when the group menu is available and negate it.

Remaining tasks

Code

User interface changes

New condition plugin

Issue fork group_content_menu-3543600

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

rodrigoaguilera created an issue. See original summary.

rodrigoaguilera opened merge request !56

rodrigoaguilera’s picture

rodrigoaguilera
Primary language Spanish
Location Madrid
commented
Status: Active » Needs review

Please review

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs review » Needs work

Left some feedback on the MR.

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented

Naming is hard. I'm not sure the name of the visibility control makes sense. Is this a filter by menu type plugin?

rodrigoaguilera’s picture

rodrigoaguilera
Primary language Spanish
Location Madrid
commented
Status: Needs work » Needs review

Thanks for reviewing.

I added a description to the group type selection to hopefully make it more clear.
Added more info to the comment of the getPluginIdWithType() method and addressed the rest of the feedback

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs review » Fixed

Thanks for the contributions here.

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.

Status: Fixed » Closed (fixed)

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