? ? Pending

User tests: Successful: Unsuccessful:

avatar chmst
chmst
10 Sep 2018

Pull Request for Issue #22093 - Another possible solution
It replaces my former PR #20787 which can be closed.

Summary of Changes

The link to the respective menuitems is moved to an own column.

Testing Instructions

Compare menu overview before and after patch. Make sure that links go to the correct menu items and that ACL permissions are respected

Actual vs. Expected result

menu-before-after

Documentation Changes Required

yes

avatar chmst chmst - open - 10 Sep 2018
avatar chmst chmst - change - 10 Sep 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Sep 2018
Category Administration com_menus Language & Strings
avatar chmst chmst - change - 10 Sep 2018
Labels Added: ? ?
avatar joomla-cms-bot joomla-cms-bot - change - 11 Sep 2018
Category Administration com_menus Language & Strings Unit Tests Administration com_admin SQL Postgresql com_associations com_banners com_cache com_categories com_contact com_content com_contenthistory com_csp
avatar brianteeman
brianteeman - comment - 11 Sep 2018

can you updqte your branch please. there shouldnt be 200 files changed for this pr.

avatar brianteeman
brianteeman - comment - 11 Sep 2018

i really like this improvement. for me it solves the usability issue and i look forward to not clicking on the wrong link.

avatar chmst
chmst - comment - 11 Sep 2018

Maybe someone can update this branch for me? I cannot do it in my repo.

avatar brianteeman
brianteeman - comment - 11 Sep 2018

Ok let me try

avatar brianteeman brianteeman - change - 11 Sep 2018
Labels Added: ?
Removed: ?
avatar brianteeman
brianteeman - comment - 12 Sep 2018

My attempt to update the branch directly here did not resolve the problem. It probably needs a --force which can't be done in the ui here. For ease I would suggest creating a new pr on the current branch and applying your changes there - iirc it was just two files.

avatar brianteeman brianteeman - change - 12 Sep 2018
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2018-09-12 09:48:17
Closed_By brianteeman
avatar brianteeman brianteeman - close - 12 Sep 2018

Add a Comment

Login with GitHub to post a comment