? ? Pending

User tests: Successful: Unsuccessful:

avatar N6REJ
N6REJ
21 Sep 2019

Pull Request for Issue # .

Summary of Changes

changes z-index so dropdowns go OVER the content below

Testing Instructions

install latest 4.0-dev.
go to components and click on the cog on "tags"
dropdown will be partially hidden.
install patch
run node build.js --compile-css
dropdown now fully visible

Expected result

image
image

Actual result

image
image

Documentation Changes Required

None

avatar N6REJ N6REJ - open - 21 Sep 2019
avatar N6REJ N6REJ - change - 21 Sep 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Sep 2019
Category Administration Templates (admin)
avatar N6REJ N6REJ - change - 21 Sep 2019
Title
dropdowns don't drop OVER the items below
[4.0] Dropdowns don't drop OVER the items below
avatar N6REJ N6REJ - edited - 21 Sep 2019
avatar rdeutz rdeutz - change - 28 Sep 2019
Labels Added: ?
avatar Thomsterdam
Thomsterdam - comment - 1 Oct 2019

I tested the patch and it does not have the required effect.

The issue was not identical to what is described above: The Tags drop-down showed correct: it dropped down over the grey area but the Smart Search drop-down showed as above in the issue.

I tested in both Firefox 69.0.1 (Linux) and Chromium 77.0.3865.90 (Linux) and the results are the same. See picture. So after applying the patch, the issue is NOT fixed.

screen shot 2019-10-01 at 12 08 29


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

avatar Thomsterdam Thomsterdam - test_item - 1 Oct 2019 - Tested unsuccessfully
avatar Thomsterdam
Thomsterdam - comment - 1 Oct 2019

I have tested this item ? unsuccessfully on 0ddb2f3

I tested the patch and it does not have the required effect.

The issue was not identical to what is described above: The Tags drop-down showed correct: it dropped down over the grey area but the Smart Search drop-down showed as above in the issue.

I tested in both Firefox 69.0.1 (Linux) and Chromium 77.0.3865.90 (Linux) and the results are the same. See picture. So after applying the patch, the issue is NOT fixed.


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

avatar richard67
richard67 - comment - 1 Oct 2019

Testing instructions are not complete, it needs to run “npm install” after having applied the patch. Or am I wrong?

avatar brianteeman
brianteeman - comment - 1 Oct 2019

Yes the test instructions are wrong

No need to do a full npm install - it is enough to run
node build.js --compile-css

This will compile all the scss defined files and also create a minified version of the css

avatar Quy Quy - test_item - 1 Oct 2019 - Tested successfully
avatar Quy
Quy - comment - 1 Oct 2019

I have tested this item successfully on 0ddb2f3


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

avatar Quy
Quy - comment - 1 Oct 2019

I have tested this item successfully on 0ddb2f3


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

avatar richard67
richard67 - comment - 1 Oct 2019

Yes, node build.js --compile-css is absolutely sufficient ... but npm install is easier to remember for me ;-)

avatar richard67 richard67 - test_item - 1 Oct 2019 - Tested successfully
avatar richard67
richard67 - comment - 1 Oct 2019

I have tested this item successfully on 0ddb2f3


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

avatar richard67
richard67 - comment - 1 Oct 2019

@Thomsterdam Please change your test result in the issue tracker to "Not tested" so maintainers don't get confused by the negative test (which was wrong due to wrong testing instructions).

avatar Quy Quy - change - 1 Oct 2019
Status Pending Ready to Commit
avatar Quy
Quy - comment - 1 Oct 2019

RTC


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

avatar N6REJ
N6REJ - comment - 2 Oct 2019

do I need to change something guys?

avatar N6REJ N6REJ - change - 2 Oct 2019
The description was changed
avatar N6REJ N6REJ - edited - 2 Oct 2019
avatar wilsonge wilsonge - change - 16 Oct 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-10-16 20:59:46
Closed_By wilsonge
Labels Added: ?
avatar wilsonge
wilsonge - comment - 16 Oct 2019

Thanks!

Add a Comment

Login with GitHub to post a comment