NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
3 Feb 2020

PR for #26652

Steps to reproduce the issue

Add a module to a dashboard on a position != cpanel.
Use chrome "well".
The module is rendered on the dashboard. And the cog for edit / unpublish appears.
Now click unpublish

Expected result

The module is unpublished

Actual result

Nothing happens

After PR

Module is unpublished as expected

Testing Notes

As this is a javascript change either npm i or node build.js compile-js are required after applying the patch

avatar joomla-cms-bot joomla-cms-bot - change - 3 Feb 2020
Category JavaScript Repository NPM Change
avatar brianteeman brianteeman - open - 3 Feb 2020
avatar brianteeman brianteeman - change - 3 Feb 2020
Status New Pending
avatar chmst chmst - test_item - 3 Feb 2020 - Tested successfully
avatar chmst
chmst - comment - 3 Feb 2020

I have tested this item successfully on 6f013c8


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

avatar drmenzelit drmenzelit - test_item - 3 Feb 2020 - Tested successfully
avatar drmenzelit
drmenzelit - comment - 3 Feb 2020

I have tested this item successfully on 6f013c8


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

avatar infograf768
infograf768 - comment - 3 Feb 2020

I am afraid this only solves the issue when the module is displayed in dashboards.
Setting the module to position bottom also displays it all over backend.

Example:
Screen Shot 2020-02-03 at 10 14 39

Not only the cog and its dropdown is not displayed at should but the event is not attached to the unpublish.

avatar chmst
chmst - comment - 3 Feb 2020

@infograf768 You are right but this is another issue.

avatar infograf768
infograf768 - comment - 3 Feb 2020

Adding in index.php
HTMLHelper::_('script', 'com_cpanel/admin-cpanel-default.min.js', array('version' => 'auto', 'relative' => true));
will solve the event but not the placement of the cog and its dropdown.

EDIT: cog and dropdown would in this case be solved by adding a class. Here for LTR

.module-actions {
    position: absolute;
    top: 0;
    right: 0;
    z-index: auto;
}
avatar brianteeman
brianteeman - comment - 3 Feb 2020

That is another issue. Please create a new issue for that

avatar infograf768 infograf768 - change - 3 Feb 2020
Labels Added: NPM Resource Changed ?
avatar brianteeman
brianteeman - comment - 3 Feb 2020

The appveyor error is nothing to do with this pr and is a general appveyor failure

avatar wilsonge wilsonge - change - 9 Feb 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-02-09 16:54:31
Closed_By wilsonge
avatar wilsonge wilsonge - close - 9 Feb 2020
avatar wilsonge wilsonge - merge - 9 Feb 2020
avatar wilsonge
wilsonge - comment - 9 Feb 2020

Thanks!

avatar brianteeman
brianteeman - comment - 9 Feb 2020

pleasure

Add a Comment

Login with GitHub to post a comment