![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 13 years ago
Closed 13 years ago
#26140 closed defect (bug) (fixed)
Adjust thickbox positioning code for new toolbar height
| Reported by: | SergeyBiryukov | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.8 |
| Component: | Administration | Version: | 3.8 |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | Focuses: |
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
@
13 years ago
How about we adjust it to use the dynamic values as such:
parseInt( jQuery('#wpadminbar').css('height'), 10 );
#3
@
13 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.