? ? ? Success

User tests: Successful: Unsuccessful:

avatar Devportobello
Devportobello
9 Sep 2016

Context

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.

Testing Instructions

  • Create a menu with some items, hide some with the param: "Display in Menu"
  • Take a look at the list of items, unable to know who is hidden or not

Patch

  • Now you can know hidden or not ### Work in progress

i dunno if the icon is at the right place (before title)

avatar Devportobello Devportobello - open - 9 Sep 2016
avatar Devportobello Devportobello - change - 9 Sep 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Sep 2016
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 9 Sep 2016
Category Administration Components
avatar Devportobello Devportobello - change - 9 Sep 2016
The description was changed
avatar Devportobello Devportobello - edited - 9 Sep 2016
avatar jeckodevelopment jeckodevelopment - change - 9 Sep 2016
Labels Added: ?
avatar brianteeman
brianteeman - comment - 9 Sep 2016

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

avatar Bakual
Bakual - comment - 9 Sep 2016

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.

avatar brianteeman
brianteeman - comment - 9 Sep 2016

That would work too

avatar Devportobello
Devportobello - comment - 12 Sep 2016

@brianteeman @Bakual
Ye i agree with you
Label will add a new text translation, adding "JHIDDEN" in "en-GB.ini" is the right approach ?

avatar Bakual
Bakual - comment - 12 Sep 2016

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.

9047df2 12 Sep 2016 avatar Devportobello ux
avatar joomla-cms-bot joomla-cms-bot - change - 12 Sep 2016
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 12 Sep 2016
Category Administration Components Administration Components Language & Strings
avatar Devportobello Devportobello - change - 12 Sep 2016
Labels Added: ?
avatar brianteeman brianteeman - test_item - 12 Sep 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 12 Sep 2016

I have tested this item successfully on 0b5251d


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

ba68b2d 13 Sep 2016 avatar Devportobello cs
avatar gwsdesk gwsdesk - test_item - 18 Sep 2016 - Tested successfully
avatar gwsdesk
gwsdesk - comment - 18 Sep 2016

I have tested this item successfully on ba68b2d


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

avatar conconnl conconnl - test_item - 4 Nov 2016 - Tested successfully
avatar conconnl
conconnl - comment - 4 Nov 2016

I have tested this item successfully on ba68b2d

Tested successfully at PBF NL


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

avatar roland-d roland-d - test_item - 4 Nov 2016 - Tested successfully
avatar roland-d
roland-d - comment - 4 Nov 2016

I have tested this item successfully on ba68b2d

After applying the patch the hidden notice shows behind the menu item.


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

avatar roland-d roland-d - change - 4 Nov 2016
The description was changed
Status Pending Ready to Commit
Labels
avatar roland-d
roland-d - comment - 4 Nov 2016

Setting to RTC


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

avatar roland-d roland-d - change - 4 Nov 2016
Labels
avatar roland-d roland-d - edited - 4 Nov 2016
avatar brianteeman brianteeman - change - 4 Nov 2016
Labels Added: ?
Removed: ?
avatar brianteeman brianteeman - change - 4 Nov 2016
Milestone Added:
avatar roland-d roland-d - change - 4 Nov 2016
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
avatar roland-d roland-d - close - 4 Nov 2016
avatar roland-d roland-d - merge - 4 Nov 2016
avatar roland-d roland-d - reference | adb8532 - 4 Nov 16
avatar roland-d roland-d - merge - 4 Nov 2016
avatar roland-d roland-d - close - 4 Nov 2016
avatar Devportobello Devportobello - head_ref_deleted - 4 Nov 2016

Add a Comment

Login with GitHub to post a comment