? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
10 Sep 2019

Pull Request for Issue #25993.

Summary of Changes

This removes incorrectly place module action icon from mod_submenu and other code mistaking submenu items for modules.

Testing Instructions

View System Dashboard.

Expected result

Still works but no more misleading module action next to submenu items.

Documentation Changes Required

?‍♂

avatar SharkyKZ SharkyKZ - open - 10 Sep 2019
avatar SharkyKZ SharkyKZ - change - 10 Sep 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Sep 2019
Category Modules Administration
avatar SharkyKZ SharkyKZ - change - 10 Sep 2019
Labels Added: ?
avatar SharkyKZ SharkyKZ - change - 10 Sep 2019
The description was changed
avatar SharkyKZ SharkyKZ - edited - 10 Sep 2019
avatar infograf768
infograf768 - comment - 10 Sep 2019

Tested
The problem here is that we can't edit the "real" module anymore as the cog is no more present for default presets.
I also failed solving this to get a unique cog there and not in child.
It works though when we create a submenu, even with a preset

Examples:
Default Content Dashboard
Screen Shot 2019-09-10 at 09 21 11

Modified Component Dashboard (Title set to Show)
Screen Shot 2019-09-10 at 09 22 17

avatar SharkyKZ
SharkyKZ - comment - 10 Sep 2019

Module action buttons belong in the chrome, not in module layout. System Dashboard module is using none chrome which doesn't have action buttons.

avatar infograf768
infograf768 - comment - 10 Sep 2019

@SharkyKZ
I have been testing all dashboards, not only system.

avatar SharkyKZ
SharkyKZ - comment - 10 Sep 2019

Still applies. Use a different chrome. I know the layout is broken when using different chromes but that's out of scope of this PR.

avatar brianteeman
brianteeman - comment - 10 Sep 2019

Still applies. Use a different chrome. I know the layout is broken when using different chromes but that's out of scope of this PR.

In this case it is in scope as its not possible to see if the changes work

avatar SharkyKZ
SharkyKZ - comment - 10 Sep 2019

This removes module action buttons from items that aren't modules.

avatar brianteeman
brianteeman - comment - 10 Sep 2019

Sorry for my confusion then on the previous post

avatar infograf768
infograf768 - comment - 11 Sep 2019

I do not get any conflict here. Looks like drone is dead though.

avatar infograf768 infograf768 - change - 11 Sep 2019
Labels Added: Conflicting Files
avatar SharkyKZ SharkyKZ - change - 30 Sep 2019
Labels Removed: Conflicting Files
avatar Quy Quy - test_item - 14 Oct 2019 - Tested successfully
avatar Quy
Quy - comment - 14 Oct 2019

I have tested this item successfully on d87c51b


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

avatar SharkyKZ
SharkyKZ - comment - 15 Oct 2019

This kills the Drone ?

avatar infograf768
infograf768 - comment - 15 Oct 2019

@SharkyKZ
I suggest to redo a clean PR as it looks like drone was not triggered by this one.

avatar SharkyKZ
SharkyKZ - comment - 15 Oct 2019

Closing in favor of #26600.

avatar SharkyKZ SharkyKZ - close - 15 Oct 2019
avatar SharkyKZ SharkyKZ - change - 15 Oct 2019
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2019-10-15 08:27:04
Closed_By SharkyKZ

Add a Comment

Login with GitHub to post a comment