Secondary nav needs manipulators for access check and sort [#2614498]

Hello,

Currently "My account" is displayed to anonymous users. I attached a patch to fix it.

But I think that instead of rendering the menu that way, we should have 2 separate regions (like navigation_primary and navigation_secondary) and add the necessary classes to the blocks rendered there. It fixes this issue too and add more flexibility (ie other menus could be added to those regions). I can create a patch for that if any interest in this.

Comment	File	Size	Author
	bootstrap-fix-account-menu.patch	845 bytes	mijost

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

13 November 2015 at 02:18

mijost created an issue. See original summary.

Comment #2

markhalliwell

English

CreditAttribution: markhalliwell at Tag1 Consulting commented 13 November 2015 at 03:23

Title:	Fix account menu	» Secondary nav needs manipulators for access check and sort
Related issues:		+#2521998: Menu link weights are disregarded in main menu

But I think that instead of rendering the menu that way, we should have 2 separate regions (like navigation_primary and navigation_secondary) and add the necessary classes to the blocks rendered there. It fixes this issue too and add more flexibility (ie other menus could be added to those regions). I can create a patch for that if any interest in this.

We'd like to keep as much parity between the 7.x-3.x and 8.x-3.x branches, so this is not likely to happen until 8.x becomes more "de facto".