? NPM Resource Changed Pending

User tests: Successful: Unsuccessful:

avatar RickR2H
RickR2H
5 Aug 2021

Pull Request for Issue # .
My previous PR #34608 got reverted due to missing styling #34750. In this version the underline is fixed.
For testing instruction and information, see my previous PR #34608.

Make sure all active menu items have an underline.
sub-level-underline

If the menu is in the header position make sure the Collapsible Dropdown layout is selected in the module -> Advanced tab.
metismenu

Summary of Changes

Testing Instructions

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Documentation Changes Required

avatar RickR2H RickR2H - open - 5 Aug 2021
avatar RickR2H RickR2H - change - 5 Aug 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Aug 2021
Category Front End Templates (site) NPM Change
avatar RickR2H RickR2H - change - 5 Aug 2021
Labels Added: NPM Resource Changed ?
avatar RickR2H
RickR2H - comment - 5 Aug 2021

@drmenzelit @richard67 @ricardo1709 @MSnoeren
Obviously this is still a bug that has to be fixed. Please review and test.

avatar RickR2H RickR2H - change - 9 Aug 2021
Labels Added: ?
Removed: ?
avatar RickR2H RickR2H - change - 9 Aug 2021
Labels Added: ?
Removed: ?
avatar RickR2H RickR2H - change - 9 Aug 2021
Labels Added: ?
Removed: ?
avatar RickR2H RickR2H - change - 9 Aug 2021
The description was changed
avatar RickR2H RickR2H - edited - 9 Aug 2021
avatar ricardo1709 ricardo1709 - test_item - 9 Aug 2021 - Tested successfully
avatar ricardo1709
ricardo1709 - comment - 9 Aug 2021

I have tested this item successfully on 41b77b6


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35047.

avatar MSnoeren MSnoeren - test_item - 9 Aug 2021 - Tested successfully
avatar MSnoeren
MSnoeren - comment - 9 Aug 2021

I have tested this item successfully on 41b77b6


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35047.

avatar drmenzelit
drmenzelit - comment - 11 Aug 2021

If I use a metismenu on the sidebar, the menu item with the very long title doesn't break:
image

Without this PR:
image

avatar RickR2H RickR2H - change - 11 Aug 2021
Labels Added: ?
Removed: ?
avatar RickR2H
RickR2H - comment - 11 Aug 2021

@drmenzelit Made some small changes. Could you please test again? The wrapping will always be a problem, also with the current menu. If words are too long the menu wrapper will be pushed outside of the right and left position.

avatar RickR2H
RickR2H - comment - 16 Jan 2022

Closed as having more then 3 levels deep is bad UX/UI practice!

avatar RickR2H RickR2H - change - 16 Jan 2022
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2022-01-16 15:10:34
Closed_By RickR2H
Labels Added: ?
Removed: ?
avatar RickR2H RickR2H - close - 16 Jan 2022

Add a Comment

Login with GitHub to post a comment