#26178 closed defect (bug) (fixed)
Double close button in thickbox
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.8 | Priority: | normal |
Severity: | normal | Version: | 3.8 |
Component: | Administration | Keywords: | needs-patch |
Focuses: | ui | Cc: |
Description
There is unnecessary background image in .tb-close-icon class (wp-includes/js/thickbox/thickbox.css). This causes thickbox window has double close button.
Attachments (2)
Change History (8)
#1
@
11 years ago
- Keywords needs-patch ui-focus added
- Milestone changed from Awaiting Review to 3.8
- Severity changed from trivial to normal
#2
@
11 years ago
Just wanted to add that this error isn't reproducible in OSX with Chrome 31.0.1650.57 or FireFox 25.0.1.
#3
@
11 years ago
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.
#4
@
11 years ago
- Owner set to iammattthomas
- Resolution set to fixed
- Status changed from new to closed
In 26395:
Note: See
TracTickets for help on using
tickets.
Happens if
SCRIPT_DEBUG
is false.thickbox.css
is loaded after the styles viaload-styles.php
.