jQuery Color 2.1.0 (update)

[ramoonus]

jQuery script Color has just been updated to version 2.1.0 as described on ​http://blog.jquery.com/2012/08/24/jquery-color-2-1-0/

[scribu]

Is jQuery.Color actually used in Core? I see the enqueue, but it's not tied to anything.

Related: #21206

[azaozz]

It's used in wp-lists.js to get/set background colors. Some of this functionality is now included in jQuery afaik.

[scribu]

Then shouldn't we just add it as a dependency to wp-lists.js instead of enqueuing it manually?

[nacin]

In [21632]:

Update jQuery Color Animations plugin to 2.1.0. Props gnarf. fixes #21692.

[scribu]

Why do we need the unminified version in Core?

And what about the dangling dependency?

[scribu]

21692.diff​ fixes the dependencies.

[scribu]

PS: I tried just removing wp_enqueue_script( 'jquery-color' ); and couldn't notice any errors or differences in the UI.

[nacin]

We can remove both un-minified versions and add them to the sources repository. Since we have changed from .js to .min.js, I didn't want to break any direct references to .js. But in this case, I don't care.

[nacin]

This was added when alerts used to have a color fade. See [6582] for when we transitioned from FAT to jQuery Color Animations. At the time, it should have been added as a dependency to 'common', just as it was a dependency for 'listman'.

We no longer have any animations in common.js as we no longer have fades in our alerts (removed in 2.9). We do however still animate colors when approving/unapproving/trashing/etc a comment, which is why we need it for wp-lists.

We only actually use the dimming aspects of wp-lists inside edit-comments, but someone else using and extending wp-lists (and, I really do apologize to them) might be. By removing the global enqueue, we're still saving some bytes, so let's move to wp-lists instead of edit-comments.

Anyone using color animations in the admin will lose them if they are not marking jquery-color as a dependency themselves (or enqueueing it themselves). If people complain, we can make it a dependency of 'common' as well.

[nacin]

In [21646]:

Set jQuery Color to be a dependency for wp-lists, rather than enqueueing it everywhere.

Any plugin wishing to use jQuery color animations must set jquery-color as a dependency
(or enqueue it).

Remove the un-minified version of jQuery Color.

props scribu.
fixes #21692.

[nacin]

In [21660]:

Update old files array:

• Remove old names for jQuery UI Effects files. see #21736.
• Remove un-minified version of Jcrop and jQuery Color. see #21692, #20728.
• Remove .dev.css and .dev.js. see #21633.