Double close button in thickbox

[matczar]

There is unnecessary background image in .tb-close-icon class (wp-includes/js/thickbox/thickbox.css). This causes thickbox window has double close button.

[ocean90]

Happens if SCRIPT_DEBUG is false.

thickbox.css is loaded after the styles via load-styles.php.

[kstover]

Just wanted to add that this error isn't reproducible in OSX with Chrome 31.0.1650.57 or FireFox 25.0.1.

[zerodegreeburn]

I reproduced this error in Windows 7 and 8, in Chrome, Firefox, IE and Opera (all latest versions).

In my opinion the original thickbox.css .tb-close-icon looks better. The new .tb-close-icon:before's icon is 12px tall and trying to fit into a 27px height - so it can't be properly centered (see screenshot).

Disabling .tb-close-icon's background fixes the issue, but the new psuedo element's alignment is off.

[iammattthomas]

In 26395:

Move thickbox styles from wp-admin.css to thickbox.css. Remove old png close icons. Fixes #26178.

[iammattthomas]

I've moved all the MP6-era thickbox styles out of wp-admin and into thickbox.css, while removing the old close button styles. I updated the font stack to include Open Sans and adjusted the margin on the close button so it's better-aligned than it was before.

[iammattthomas]

In 26396:

Remove the new thickbox styles from wp-admin.css. See #26178