? Pending

User tests: Successful: Unsuccessful:

avatar Bakual
Bakual
6 Mar 2017

Updates the jQuery MiniColors library to the latest version (from 2.2.3 to 2.2.4)

Summary of Changes

Just applying latest patch version

Testing Instructions

Make sure the color field (eg color picker in the protostar template style) still works as expected.

Disclaimer

Work is done by @brianteeman

avatar Bakual Bakual - open - 6 Mar 2017
avatar Bakual Bakual - change - 6 Mar 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Mar 2017
Category JavaScript
avatar Bakual Bakual - change - 6 Mar 2017
The description was changed
avatar Bakual Bakual - edited - 6 Mar 2017
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 6 Mar 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 6 Mar 2017

I have tested this item successfully on 181c76b

it works but:


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

avatar infograf768
infograf768 - comment - 6 Mar 2017

that means not good as is.

avatar infograf768 infograf768 - test_item - 6 Mar 2017 - Tested unsuccessfully
avatar infograf768
infograf768 - comment - 6 Mar 2017

I have tested this item ? unsuccessfully on 181c76b


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 6 Mar 2017 - Tested unsuccessfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 6 Mar 2017

I have tested this item ? unsuccessfully on 181c76b


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

avatar Bakual
Bakual - comment - 6 Mar 2017

Ah I see, last time when the picker was updated the CSS was left original. Issue is that they rewrote the CSS for Bootstrap support somewhere around their version 2.1.0
When we revert the change in the CSS it works fine again.

avatar brianteeman
brianteeman - comment - 9 Mar 2017

So should I remove our hacks to the css?

avatar Bakual
Bakual - comment - 9 Mar 2017

We can either revert the CSS to the one in staging, which should work from what I saw.
Or we can try to copy the BS2 CSS rules into their current CSS and see how that works out. But that would mean hacking a 3rd party library to a degree.

avatar Bakual
Bakual - comment - 31 Mar 2017

I'm closing this PR as @brianteeman is now able to create his own PRs again ?

avatar Bakual Bakual - change - 31 Mar 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-03-31 13:19:38
Closed_By Bakual
Labels Added: ?
avatar Bakual Bakual - close - 31 Mar 2017
avatar brianteeman
brianteeman - comment - 31 Mar 2017

thanks @Bakual in this case i am going to abandon this pr

Add a Comment

Login with GitHub to post a comment