Press This focus/active style broken

[johnbillion]

Most likely due to changes made as part of #21598, the Press This button has lost its background style when being dragged.

[helenyhou]

Tested, looks good.

[nacin]

In [21832]:

Fix Press This bookmarklet focus and active styles. props SergeyBiryukov, fixes #21850.

[ocean90]

Active status is still broken. 21850.2.patch​ fixes it.

[davidcruz]

Replying to ocean90:

Active status is still broken. 21850.2.patch​ fixes it.

I tested your patch in a trunk deploy but it didn't fix the issue because you forgot to also include the fix in the minified CSS file.

I attached an expanded version of your patch (21850.2b.patch​) to apply the change to the minified file.

[SergeyBiryukov]

Replying to davidcruz:

I tested your patch in a trunk deploy but it didn't fix the issue because you forgot to also include the fix in the minified CSS file.

There's need to patch minified files, a post-commit bot takes care of it.

To test CSS or JS patches, you should add SCRIPT_DEBUG constant to your wp-config.php file:
​http://codex.wordpress.org/Debugging_in_WordPress#SCRIPT_DEBUG

[davidcruz]

Oh OK, sorry =(

I asked some folks at WordCamp to see if there was some building procedure to minify but seems nobody knew about post-commit hook.
Thank for info.

Replying to SergeyBiryukov:

Replying to davidcruz:

I tested your patch in a trunk deploy but it didn't fix the issue because you forgot to also include the fix in the minified CSS file.

There's no need to patch minified files, a post-commit bot takes care of it.

To test CSS or JS patches, you should add SCRIPT_DEBUG constant to your wp-config.php file:
​http://codex.wordpress.org/Debugging_in_WordPress#SCRIPT_DEBUG

[nacin]

In [22165]:

Fix the Press This bookmarklet's active style. props ocean90, fixes #21850.