? ? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
30 Dec 2015

Description

This is PR takes the contributions from #8814 and adds the ability to exclude menu items from displaying in frontend menus.

For more info see #8814

How to test

  1. Fresh Joomla install with multilanguage.
  2. Apply this patch
  3. Go to a menu item that is displayed in the frontend main menu, separator "Link Type" and set the option "Display in Menu Modules" to "No" to exclude the menu item from displaying in the frontend menu.
  4. Check in the frontend it those items are not displayed in the main menu.

Observations

Suggestions are welcome.

avatar andrepereiradasilva andrepereiradasilva - open - 30 Dec 2015
avatar andrepereiradasilva andrepereiradasilva - change - 30 Dec 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 30 Dec 2015
Labels Added: ? ?
avatar andrepereiradasilva andrepereiradasilva - change - 31 Dec 2015
Title
mod_menu: Allowing to exclude menu items from displaying in frontend menus (in menu item)
Allowing to exclude menu items from displaying in frontend menus (in menu item)
avatar andrepereiradasilva
andrepereiradasilva - comment - 31 Dec 2015

@justinherrin done.
Can you test if everything ok now?

avatar justinherrin justinherrin - test_item - 31 Dec 2015 - Tested successfully
avatar justinherrin
justinherrin - comment - 31 Dec 2015

I have tested this item :white_check_mark: successfully on 3d547b6

I have successfully tested this PR by using a bunch of different menu types (item alias, menu heading, text separator, external url, articles, tags, etc).


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

avatar Fedik
Fedik - comment - 31 Dec 2015

just curious (cannot test now) how it will work if parent menu is "hidden" but the children is visible? ... on first look it will fail

avatar andrepereiradasilva
andrepereiradasilva - comment - 31 Dec 2015

@Fedik, you're right i tested right now.

If you hide the parent menu item that menu subitems will show, and they shouldn't.
Have to correct that.

avatar joomla-cms-bot
joomla-cms-bot - comment - 31 Dec 2015

This PR has received new commits.

CC: @justinherrin


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

avatar anibalsanchez anibalsanchez - test_item - 2 Jan 2016 - Tested successfully
avatar anibalsanchez
anibalsanchez - comment - 2 Jan 2016

I have tested this item :white_check_mark: successfully on 237811c

Test OK, simple menu item and parent menu item tested


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

avatar NLRoosj NLRoosj - test_item - 10 Jan 2016 - Tested successfully
avatar NLRoosj
NLRoosj - comment - 10 Jan 2016

I have tested this item :white_check_mark: successfully on 237811c


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

avatar brianteeman brianteeman - change - 12 Jan 2016
Status Pending Ready to Commit
Labels
avatar brianteeman
brianteeman - comment - 12 Jan 2016

RTC - thanks


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

avatar joomla-cms-bot joomla-cms-bot - change - 12 Jan 2016
Labels Added: ?
avatar andrepereiradasilva
andrepereiradasilva - comment - 12 Jan 2016

@anibalsanchez @NLRoosj thanks for testing

avatar rdeutz rdeutz - change - 13 Jan 2016
Milestone Added:
avatar izharaazmi
izharaazmi - comment - 20 Jan 2016

Currently we suggest people to create separate menu type and add such menu items which they don't want to show. This PR sounds better than that.
@andrepereiradasilva Can you please elaborate over what benefits this is supposed to give us? As you are the author, you have better insights than me.

avatar andrepereiradasilva
andrepereiradasilva - comment - 20 Jan 2016

In my view the benefits are:
1. You don't have to create an "hidden menu" for hidden menu items. You just put then in the menu you want and then hide the items you want with this option.
2. You can now hide only second, third, etc level menu items levels from the menu keeping the information architecture itself organized (breadcrumb, sitemap, sef uri's, etc) as it should.
3. Usability. IMHO it's simpler for a user to understand this behavior.
ex: A user is in a menu item that he don't want to appear in the site menu ... he just select this new option in the menu item itself and hide it. Simple as that. No tricks
4. BC. You can still create an "hidden menu" item as you do before, if you wish so.

avatar izharaazmi
izharaazmi - comment - 20 Jan 2016

Great! Point no. 2 is what cheers me.

Can you also add Hide and Unhide buttons in the list view just like the publish/unpublish behaviour for quick access. Better if we can add a column too in the list view for this.

avatar anibalsanchez
anibalsanchez - comment - 20 Jan 2016

In fact, I think this option is simpler than separated "hidden menu". A "hidden menu" is a trick that usually leads to have many assorted menu items, with no cohesion.

avatar brianteeman brianteeman - test_item - 25 Jan 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 25 Jan 2016

I have tested this item :white_check_mark: successfully on 237811c

Does what it says - a useful addition
Made some suggestions to simplify the text a bit and make it better english
I do still wonder if it would not be better as Show/Hide instead of Yes/No - that would seem more appropriate to me


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

avatar izharaazmi
izharaazmi - comment - 25 Jan 2016

I think this should be in line with properties like published. Params is
not much appropriate IMHO.
Also see my other comments whether those make sense.

avatar joomla-cms-bot
joomla-cms-bot - comment - 25 Jan 2016

This PR has received new commits.

CC: @anibalsanchez, @brianteeman, @justinherrin, @NLRoosj


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 25 Jan 2016

@izharaazmi i tought about your comments,

I think this should be in line with properties like published. Params is not much appropriate IMHO.

I see several problems with that:
1. A menu item can be Published and hidden, or Not Published and hidden so is not a state like Published, Not Published, Trashed.
2. Consistensy: none of the other components have an hidden state.

Can you also add Hide and Unhide buttons in the list view just like the publish/unpublish behaviour for quick access. Better if we can add a column too in the list view for this.

Like the "featured" button in the articles list view?
I guess it can be done with a new field in the table. But is it really worth it? Will it not create some confusion to new users?
IMHO, that is to much emphasis for a option to hide a menu item in the menu modules.

@brianteeman

I do still wonder if it would not be better as Show/Hide instead of Yes/No - that would seem more appropriate to me

I put "Yes/No" to use the same as "Add Menu Title" "Yes/No" in the same tab.
But if you guys think "Show/Hide" (with a colour dropdown?) is better i can change it.

avatar izharaazmi
izharaazmi - comment - 26 Jan 2016

Well, sounds great to me.

I still think three things will be very useful for the common/average users (priority top to bottom):

  1. Bulk action (using checkboxes) to hide/unhide items.
  2. A way to quickly say which menu item is hidden or not, without needing to open/edit and lookup each one.
  3. Filter by visibility.

Tell me if you want me to do the PR for (any of) these.

avatar izharaazmi
izharaazmi - comment - 26 Jan 2016

Regarding #8814. I see this one has also great usability. Assume some case when we want to render same menu and show different set of menu items in different menu module positions!
With that we could override/inherit the selections at will.
Can we have a second opinion @brianteeman?

Great work @andrepereiradasilva.

avatar brianteeman
brianteeman - comment - 26 Jan 2016

Let's stick to the subject of this PR and not discuss other ones

avatar andrepereiradasilva
andrepereiradasilva - comment - 26 Jan 2016

I still think three things will be very useful for the common/average users (priority top to bottom):
Bulk action (using checkboxes) to hide/unhide items.
A way to quickly say which menu item is hidden or not, without needing to open/edit and lookup each one.
Filter by visibility.

@izharaazmi I will not do more PR regarding this matter (i have other PR to do).

avatar wilsonge wilsonge - change - 26 Jan 2016
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2016-01-26 10:44:11
Closed_By wilsonge
avatar wilsonge wilsonge - close - 26 Jan 2016
avatar wilsonge wilsonge - change - 26 Jan 2016
Milestone Added:
avatar wilsonge wilsonge - change - 26 Jan 2016
Milestone Removed:
avatar wilsonge
wilsonge - comment - 26 Jan 2016

Merged for 3.5.0 - thanks @andrepereiradasilva :)

avatar joomla-cms-bot joomla-cms-bot - change - 26 Jan 2016
Labels Removed: ?
avatar andrepereiradasilva andrepereiradasilva - head_ref_deleted - 26 Jan 2016

Add a Comment

Login with GitHub to post a comment