? Pending

User tests: Successful: Unsuccessful:

avatar ciar4n
ciar4n
17 Aug 2018

Pull Request for Issue #21639 .

Summary of Changes

#21615 removed some required functions and variables which killed the color variations in the switcher. This PR fixes that.

Testing Instructions

Apply PR and run node build.js --compile-ce

Check switcher colors (article edit)

Before

See #21639

After

image

Documentation Changes Required

avatar ciar4n ciar4n - open - 17 Aug 2018
avatar ciar4n ciar4n - change - 17 Aug 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Aug 2018
Category Repository
avatar infograf768
infograf768 - comment - 17 Aug 2018

NOT for PatchTester users. Requires npm install (or node)

avatar infograf768 infograf768 - test_item - 17 Aug 2018 - Tested successfully
avatar infograf768
infograf768 - comment - 17 Aug 2018

I have tested this item successfully on 70db24f


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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Aug 2018

@infograf768 thanks for "NOT for PatchTester users. Requires npm install (or node)".

avatar brianteeman brianteeman - test_item - 17 Aug 2018 - Tested successfully
avatar brianteeman
brianteeman - comment - 17 Aug 2018

I have tested this item successfully on 70db24f


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 17 Aug 2018
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Aug 2018

Ready to Commit after two successful tests.

avatar wilsonge wilsonge - change - 17 Aug 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-08-17 14:25:37
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 17 Aug 2018
avatar wilsonge wilsonge - merge - 17 Aug 2018
avatar wilsonge
wilsonge - comment - 17 Aug 2018

Thanks!

avatar ciar4n
ciar4n - comment - 17 Aug 2018

Thanks for the tests!

Add a Comment

Login with GitHub to post a comment