![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 13 years ago
Closed 12 years ago
#26745 closed defect (bug) (fixed)
Awkward placement of more information plugin screen
| Reported by: | qBart | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.9 |
| Component: | Plugins | Version: | 3.8 |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | Focuses: | ui, administration |
Description
The placement of the 'More information' screen of plugins appears awkward on smaller screens (I have 13 inch). A part gets cut of. It should appear higher, which is also possible, see screenshot.
I'm using OS X 10.9 with Firefox 26.0.
Screenshot: http://cl.ly/image/3f3B0Q0Z143j
Attachments (4)
Change History (18)
#3
@
13 years ago
Sorry to hear that.
Safari on OS X has the same problem for me (see png screenshot).
#4
@
13 years ago
- Resolution → invalid
- Status new → closed
I tried a different WordPress installation, and it worked fine.
Sorry, must be a plugin!
Closing the issue.
#6
@
12 years ago
- Keywords needs-testing needs-patch added
- Resolution invalid
- Status closed → reopened
Hi, i am author of plugin mentioned on this issue.
After my test i found that is NOT related to my code but to this class: wp_admin_bar_class
This is the cause of the problem:
add_filter( 'wp_admin_bar_class', '__return_false' );
Read more here http://wordpress.org/support/topic/ui-problem-caused-by-plugin-bug/ Thanks!
Sorry for this ... :)
#7
@
12 years ago
- Component Administration → Plugins
- Focuses administration added
- Milestone → Awaiting Review
#9
@
12 years ago
- Keywords has-patch commit added; needs-testing needs-patch removed
- Milestone Awaiting Review → 3.9
So, the plugin removes toolbar in the admin. We don't support that: #19685, #19766.
However, it uncovered a legitimate issue in our thickbox positioning code introduced in [26323].
We use $('body.admin-bar').length to detect toolbar presence, and jQuery('#wpadminbar').css('height') to get its height, so adminbar_height ends up being NaN if #wpadminbar is not present.
We should consistently use #wpadminbar in both places. See 26745.patch.
#10
@
12 years ago
I have modified this files according patch:
/wp-admin/js/media-upload.js
/wp-admin/js/plugin-install.js
/wp-admin/js/theme.js
Tested on WordPress 3.8.0 and 3.8.1: work for me! (testing is needed from other)
Thanks to SergeyBiryukov :-),
P.S. This is my "first" WordPress Core Bug Ticket ...
#12
@
12 years ago
- Resolution worksforme
- Status closed → reopened
Tickets are closed when a commit is made to the WordPress trunk.
This is the same as the url