Opened 11 years ago
Closed 11 years ago
#26745 closed defect (bug) (fixed)
Awkward placement of more information plugin screen
Reported by: | qBart | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 3.9 | Priority: | normal |
Severity: | normal | Version: | 3.8 |
Component: | Plugins | Keywords: | has-patch commit |
Focuses: | ui, administration | Cc: |
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
@
11 years ago
Sorry to hear that.
Safari on OS X has the same problem for me (see png screenshot).
#4
@
11 years ago
- Resolution set to invalid
- Status changed from new to closed
I tried a different WordPress installation, and it worked fine.
Sorry, must be a plugin!
Closing the issue.
#6
@
11 years ago
- Keywords needs-testing needs-patch added
- Resolution invalid deleted
- Status changed from closed to 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
Read more here http://wordpress.org/support/topic/ui-problem-caused-by-plugin-bug/ Thanks!
Sorry for this ... :)
#7
@
11 years ago
- Component changed from Administration to Plugins
- Focuses administration added
- Milestone set to Awaiting Review
#9
@
11 years ago
- Keywords has-patch commit added; needs-testing needs-patch removed
- Milestone changed from Awaiting Review to 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
@
11 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
@
11 years ago
- Resolution worksforme deleted
- Status changed from closed to reopened
Tickets are closed when a commit is made to the WordPress trunk.
This is the same as the url