Problem/Motivation

See core issue: #1869638: Make the menu shown in the administration menu tray configurable

The patch in that issue adds a new admin_tray.menu_name variable in toolbar.settings config, for configuring which menu gets used in the toolbar administration tray.

Proposed resolution

The core patch makes a change to ToolbarController::preRenderAdministrationTray() to use a new admin_tray.menu_name variable in toolbar.settings config. With this variable and the Drupal configuration override functionality the amswap module codebase would be much more cleaner, and more compatible with other modules and themes.

User interface changes

none

API changes

The menu pre render call is removed.
Configuration override is used to replace the menu.

Data model changes

none

Comments

Pasqualle created an issue. See original summary.

pasqualle’s picture

Status: Active » Postponed
StatusFileSize
new4.37 KB

Patch attached. Postponing as dependent on #1869638

droces’s picture

Yes, that core patch would be great, and would greatly simplify this module. But it doesn't look like it's going to happen any time soon though...

ricovandevin’s picture

The patch does not apply anymore in version 3.1.0.

arrow’s picture

Version: 3.0.0 » 3.1.0
StatusFileSize
new2.79 KB
new3.69 KB

Rerolled for 3.1.0 and added check to allow fallback to existing behavior.

klonos’s picture

Issue summary: View changes

...linkifying the core issue number in the issue summary.

cedewey’s picture

Version: 3.1.0 » 3.2.x-dev
pasqualle’s picture

The patch does not apply

pasqualle’s picture

StatusFileSize
new2.96 KB

reroll