Opened 11 years ago
Closed 11 years ago
#26140 closed defect (bug) (fixed)
Adjust thickbox positioning code for new toolbar height
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.8 | Priority: | normal |
Severity: | normal | Version: | 3.8 |
Component: | Administration | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
We have a code in several files that makes sure whenever thickbox is used, it's doesn't overlap with Toolbar.
Since the Toolbar height changed from 28px to 32px, we should adjust the code.
Attachments (2)
Change History (6)
#2
@
11 years ago
How about we adjust it to use the dynamic values as such:
parseInt( jQuery('#wpadminbar').css('height'), 10 );
#3
@
11 years ago
26140.2.patch uses jQuery('#wpadminbar').css('height')
.
Also fixes the issue with thickbox positioning on Manage Themes screen (ticket:26078:3).
Note: See
TracTickets for help on using
tickets.
Note that the height of the bar changes at narrower screen sizes. See George's comment here.