User tests: Successful: Unsuccessful:
After #45352 is merged, more issues with the menu can be fixed. This PR now fixes two issues:
This is a bit tricky, as we have to kind of fake a deep hierarchy in the backend. So either create your own custom backend menu or we use a little trick to inject a deeper hierachy:
administrator/modules/mod_menu/src/Menu/CssMenu.phpelseif ($item->element === 'com_contact' && isset($query['view']) && $query['view'] === 'contacts') {
MenusHelper::loadPreset('content', false, $item);
}
This will load the content menu below the com_contact menu.
- After refreshing the page, the selected active menu item is not opened
- Styling is not correct
Please select:
Documentation link for docs.joomla.org:
No documentation changes for docs.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | Modules Administration JavaScript Repository NPM Change |
| Labels |
Added:
NPM Resource Changed
PR-6.0-dev
|
||
I have tested this item ✅ successfully on 88074e3
I have tested this successfully. Thanks @bembelimen for the work!
Hi @bembelimen, I have the BEFORE condition (including the JS console error and incorrect styling), after placing that code after line 423, but I have the standard design for that area and not what you show in your AFTER image - the good news there is NO JS console error which I have in the BEFORE situation once the PR is applied - so I think that I should show you: