? Failure

User tests: Successful: Unsuccessful:

avatar ciar4n
ciar4n
31 May 2017

Pull Request for Issue # .

Summary of Changes

This component sidebar items for articles are identical to the main nav items. This PR removes the component sidebar.

Testing Instructions

Navigate through all items under 'Content' and ensure the sidebar is removed and table displaying full width. Check that other list views are uneffected.

Before

content-sb1

After

content-sb2

Documentation Changes Required

avatar ciar4n ciar4n - open - 31 May 2017
avatar ciar4n ciar4n - change - 31 May 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 31 May 2017
Category Administration com_categories com_content com_fields Layout
avatar laoneo
laoneo - comment - 31 May 2017

Do you plan to do the rest as well like com_contact?
image

avatar ciar4n
ciar4n - comment - 31 May 2017

I figured only components where the component sidebar is identical to the main navigation. Where there is a difference (eg. com_contacts) the sidebar can remain.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 31 May 2017

I have tested this item successfully on a43eb90


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 31 May 2017 - Tested successfully
avatar laoneo
laoneo - comment - 31 May 2017

So I guess we need to find out why there is a difference on the other components. Personally I think it should be avoided to have two menus there.

avatar brianteeman
brianteeman - comment - 1 Jun 2017

the only reason they are different (in core) was because we didnt insist that the sidebar was the same as the submenu.

with the new ui then that argument is mute and for contacts as an example everything in the sudebar can be moved to the menu

avatar brianteeman
brianteeman - comment - 1 Jun 2017

I have tested this item successfully on a43eb90


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

avatar brianteeman brianteeman - test_item - 1 Jun 2017 - Tested successfully
avatar Bakual
Bakual - comment - 1 Jun 2017

It's usually the fields links that are additionally in the sidebar, but are not in the menu.
This is because those links have a conditional (can be turned off in the components options) and that doesn't work for the menuitems.
So if we drop the parameter for the fields links, we can add those links to the menu and get rid of the sidebar in most components. Or we add the possibility for menuitems with conditionals. Personally I'd prefer the first option. ?

avatar brianteeman
brianteeman - comment - 1 Jun 2017

This is because those links have a conditional (can be turned off in the components options) and that doesn't work for the menuitems.

check content in 3.7 - disable fields and they are gone from the menu :)

or is that because of the hardcoded nature of the content menu where as components is more dynamic

avatar Bakual
Bakual - comment - 1 Jun 2017

Yep, the content one is different. That's the only place where it works due to its hardcoded nature.

avatar ciar4n ciar4n - change - 4 Jun 2017
Labels Added: ?
avatar ciar4n
ciar4n - comment - 4 Jun 2017

Conflicts fixed

avatar brianteeman brianteeman - change - 8 Jun 2017
Milestone Added:
avatar brianteeman brianteeman - change - 8 Jun 2017
Milestone Added:
avatar wilsonge wilsonge - change - 18 Jun 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-06-18 14:36:17
Closed_By wilsonge
avatar wilsonge wilsonge - close - 18 Jun 2017
avatar wilsonge wilsonge - merge - 18 Jun 2017

Add a Comment

Login with GitHub to post a comment