Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#26745 closed defect (bug) (fixed)

Awkward placement of more information plugin screen

Reported by: qbart's profile qBart Owned by: sergeybiryukov's profile 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)

sceenshot.png (278.5 KB) - added by qBart 11 years ago.
This is the same as the url
Screen Shot 2013-12-30 at 11.29.11.pxm (944.7 KB) - added by qBart 11 years ago.
Safari sceenshot OS X
Screen Shot 2013-12-30 at 11.29.11.png (307.7 KB) - added by qBart 11 years ago.
Safari sceenshot OS X
26745.patch (1.3 KB) - added by SergeyBiryukov 11 years ago.

Download all attachments as: .zip

Change History (18)

@qBart
11 years ago

This is the same as the url

#1 @SergeyBiryukov
11 years ago

  • Component changed from General to Administration
  • Keywords ui-focus added

#2 @SergeyBiryukov
11 years ago

Could not reproduce in Firefox 26 on Windows.

@qBart
11 years ago

Safari sceenshot OS X

@qBart
11 years ago

Safari sceenshot OS X

#3 @qBart
11 years ago

Sorry to hear that.

Safari on OS X has the same problem for me (see png screenshot).

#4 @qBart
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.

#5 @ocean90
11 years ago

  • Milestone Awaiting Review deleted

#6 @sLaNGjI
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 ... :)

Version 3, edited 11 years ago by sLaNGjI (previous) (next) (diff)

#7 @SergeyBiryukov
11 years ago

  • Component changed from Administration to Plugins
  • Focuses administration added
  • Milestone set to Awaiting Review

#8 @sLaNGjI
11 years ago

Thanks you very much SergeyBiryukov :)

#9 @SergeyBiryukov
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.

Last edited 11 years ago by SergeyBiryukov (previous) (diff)

#10 @sLaNGjI
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 ...

Last edited 11 years ago by sLaNGjI (previous) (diff)

#11 @sLaNGjI
11 years ago

  • Resolution set to worksforme
  • Status changed from reopened to closed

#12 @SergeyBiryukov
11 years ago

  • Resolution worksforme deleted
  • Status changed from closed to reopened

Tickets are closed when a commit is made to the WordPress trunk.

#13 @TobiasBg
11 years ago

Looks good, Sergey. Also, jQuery('#wpadminbar') should be changed to $('#wpadminbar') for consistency.

We might even add selector caching, both for $('#wpadminbar') and $(window) in the tb_position functions, but that could go to a new ticket.

#14 @SergeyBiryukov
11 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from reopened to closed

In 27248:

Fix thickbox positioning if toolbar is not present.

fixes #26745.

Note: See TracTickets for help on using tickets.