? ? Pending

User tests: Successful: Unsuccessful:

avatar ciar4n
ciar4n
25 Feb 2020

Pull Request for Issue # .

Summary of Changes

This appears to be redundant CSS. No sign of such a class in the login markup.

Testing Instructions

Apply this patch and run node build.js --compile-css for updating the changed SCSS. Check login page displays correctly.

avatar ciar4n ciar4n - open - 25 Feb 2020
avatar ciar4n ciar4n - change - 25 Feb 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Feb 2020
Category Administration Templates (admin)
avatar Quy
Quy - comment - 25 Feb 2020

I have tested this item successfully on 77703d4


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

avatar Quy Quy - test_item - 25 Feb 2020 - Tested successfully
avatar SharkyKZ
SharkyKZ - comment - 25 Feb 2020

I have tested this item successfully on 77703d4


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

avatar SharkyKZ SharkyKZ - test_item - 25 Feb 2020 - Tested successfully
avatar SharkyKZ SharkyKZ - change - 25 Feb 2020
Status Pending Ready to Commit
Labels Added: ?
avatar SharkyKZ
SharkyKZ - comment - 25 Feb 2020

RTC.


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

avatar SharkyKZ
SharkyKZ - comment - 25 Feb 2020

RTC.


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

avatar richard67
richard67 - comment - 25 Feb 2020

@ciar4n Could you fix the conflict in file administrator/templates/atum/scss/blocks/_login.scss? Done it myself.

avatar richard67 richard67 - change - 25 Feb 2020
Labels Added: ?
avatar richard67
richard67 - comment - 25 Feb 2020

@ciar4n It seems there went something wrong when I resolved the conflict. I am sure I did it right, removed the code to be removed with this PR and not added back the code removed with the previously merged #28078, but the result looks wrong, GitHub has added back the code which had been removed with #28078 . Can it bee that I just was too fast?

Update: Solved.

avatar richard67 richard67 - change - 25 Feb 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-02-25 18:44:53
Closed_By richard67
avatar richard67 richard67 - close - 25 Feb 2020
avatar richard67 richard67 - merge - 25 Feb 2020
avatar richard67
richard67 - comment - 25 Feb 2020

Thanks.

Add a Comment

Login with GitHub to post a comment