User tests: Successful: Unsuccessful:
The new feature of the menu component allowing to show/hide a menu item from params, when we watch all items we can't know if item is hidden or not.
Patch
i dunno if the icon is at the right place (before title)
Status | New | ⇒ | Pending |
Labels |
Added:
?
|
Category | ⇒ | Administration Components |
Labels |
Added:
?
|
Instead of showing an icon (with a tooltip) maybe you can just show a small label that says "hidden" behind the title. Similar to how we show unpublished articles in frontend to the admins.
That would work too
@brianteeman @Bakual
Ye i agree with you
Label will add a new text translation, adding "JHIDDEN" in "en-GB.ini" is the right approach ?
I wouldn't make it a global string. It's only used in the menu manager. Just put it into the en-GB.com_menus.ini
and name it something like COM_MENUS_LABEL_HIDDEN
.
Labels |
Added:
?
|
Category | Administration Components | ⇒ | Administration Components Language & Strings |
Labels |
Added:
?
|
I have tested this item
I have tested this item
I have tested this item
Tested successfully at PBF NL
I have tested this item
After applying the patch the hidden notice shows behind the menu item.
Status | Pending | ⇒ | Ready to Commit |
Labels |
Setting to RTC
Labels |
Labels |
Added:
?
Removed: ? |
Milestone |
Added: |
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-11-04 11:04:44 |
Closed_By | ⇒ | roland-d |
I see what you are trying to do but I dont think this is the correct approach.
As the majority of menu items will be visible I dont see the need to always show the eye-icon
So I would probably only show the icon for items that are invisible and I would display that after the title.
Also you need some sort of tooltip/popover to explain the meaning of the icon