? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
7 May 2016

Pull Request for New Issue.

Summary of Changes

When we are in Menus -> Manage (com_menus, menus view) we have two links like this

image

But, the link behaviour is different from all other views. In all other views the title/name is the link to edit the item.

So this PR exchange the links for better UX and consistency. Also adds a "Edit" tooltip.
image

Testing Instructions

  1. Apply patch in latest staging
  2. Go to Menus -> Manage and check the links.

Votes

# of Users Experiencing Issue
0/1
Average Importance Score
1.00

avatar andrepereiradasilva andrepereiradasilva - open - 7 May 2016
avatar andrepereiradasilva andrepereiradasilva - change - 7 May 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 7 May 2016
Labels Added: ?
avatar MATsxm
MATsxm - comment - 7 May 2016

#10294 works as described but even if IMHO there IS an inconsistency having to click on Menu Type to edit the menu, I'm not sure the solution here is more consistent :wink: (what doesn't mean I have a better solution)

avatar andrepereiradasilva
andrepereiradasilva - comment - 8 May 2016

since we now have counting i propose to remove the link to the menu items.
This would be consistent with, for instance, the category alias.

avatar brianteeman brianteeman - change - 8 May 2016
Category Administration
avatar brianteeman
brianteeman - comment - 8 May 2016

I have thought about this behaviour for a long time recently as I always click on the wrong one myself. I understand why it is the way that it is but I agree that this is not consistent and should be fixed (swapped)


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 8 May 2016

ok. So test please.

avatar grhcj grhcj - test_item - 8 May 2016 - Tested successfully
avatar grhcj
grhcj - comment - 8 May 2016

I have tested this item :white_check_mark: successfully on a981021


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

avatar pe7er pe7er - test_item - 8 May 2016 - Tested successfully
avatar pe7er
pe7er - comment - 8 May 2016

I have tested this item :white_check_mark: successfully on a981021

I've tested this PR successfully. It does what it says.

However, the "manage menu items" links are used a lot more than the "edit menu settings" links.
IMHO changing them like this PR is not an improvement.


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

avatar MATsxm MATsxm - test_item - 8 May 2016 - Tested successfully
avatar MATsxm
MATsxm - comment - 8 May 2016

I have tested this item :white_check_mark: successfully on a981021

to confirm it works but I already commented I'm very skeptical


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 8 May 2016

how about adding a column with link to the menu items and removing the link from the "menu type"?

avatar andrepereiradasilva
andrepereiradasilva - comment - 8 May 2016

i can do that if it helps the skeptics :)

avatar brianteeman brianteeman - change - 8 May 2016
Status Pending Needs Review
avatar brianteeman
brianteeman - comment - 8 May 2016

Thanks for the code and the tests. I am setting this to Needs Review so that the CMS maintainers can make a decision about this changed behaviour


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 13 May 2016

clicked again on the menu name to edit the menu and was redirected to that menu menu items ... as said, i think this is not good UX

avatar andrepereiradasilva
andrepereiradasilva - comment - 31 Jul 2016

Need review ...

avatar infograf768
infograf768 - comment - 2 Aug 2016

Why not have 2 columns, one for each?

avatar andrepereiradasilva
andrepereiradasilva - comment - 2 Aug 2016

well, i don't care, if is one or two columns.

IMO this is a usability error.

In all other views when you click the item title you go to the edit page of that item, but not on this one.
This PR is to solve that usability error.

avatar andrepereiradasilva
andrepereiradasilva - comment - 5 Sep 2016

can a mantainer please take a decision here? merge or close?

avatar rdeutz
rdeutz - comment - 6 Sep 2016

I am in the same boat, I also click often on the wrong link. I don't think this here is a good solution, for making a UI change we should come up we something more brilliant. :-) We then this is something for a minor release so I am closing this and we might can ping our UI/UX group to think about it and come up with something.

Thank you all for the discussion and thanks @andrepereiradasilva for working on this.

avatar rdeutz rdeutz - change - 6 Sep 2016
Status Needs Review Closed
Closed_Date 0000-00-00 00:00:00 2016-09-06 15:40:04
Closed_By rdeutz
avatar rdeutz rdeutz - close - 6 Sep 2016
avatar joomla-cms-bot joomla-cms-bot - change - 6 Sep 2016
Category Administration Administration Components
avatar andrepereiradasilva
andrepereiradasilva - comment - 6 Sep 2016

sorry to read this decision, but it's your call.

Add a Comment

Login with GitHub to post a comment