[#25995] - [4.0][Admin menu] Allow multiple menus including expanded ones in sidebar
- Fixed in Code Base
- 27 Aug 2019
- Medium
- Build: 4.0-dev
- # 25995
- Diff
- infograf768:4.0_admin_menu_js
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
Pull Request for Issue #25953
Summary of Changes
Thanks @Fedik
The admin-menu js was only loading the metismenu event on the first occurence of a menu.
This PR also creates different ids per menu
Steps to reproduce the issue
Create a custom admin menu with no presets (we have other issues with presets and I will create another issue for that.)
Create a menu module for that menu in the "menu" position.
Make sure the module is placed AFTER the default Admin Menu module in the Manager.
Create a Heading menu item in that custom admin menu.
Create any other menu item as child of that Heading menu item.
The result is that we have both the default Admin Menu and this custom one in the menu side bar. Below, the custom admin menu starts with Global Config.
Before patch
The items with arrows in the second menu do not expand
After patch
Issue solved
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Modules Administration JavaScript Repository NPM Change |
| Labels |
Added:
NPM Resource Changed
?
|
||
At last, hound happy.... GRRRR
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25995.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25995.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25995.
| Status | Pending | ⇒ | Ready to Commit |
Status "Ready To Commit".
| Labels |
Added:
?
|
||
Thanks for this menu fix.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-08-27 11:42:43 |
| Closed_By | ⇒ | HLeithner |
Need help for hound... @Fedik @wilsonge