WordPress.org

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#6676 closed defect (bug) (fixed)

Uploader shifted to bottom of browser

Reported by: MtDewVirus Owned by:
Milestone: 2.6 Priority: normal
Severity: normal Version: 2.6
Component: General Keywords: uploader
Focuses: Cc:

Description

For whatever reason, IE 7 can report the first part of the UA string as "Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 5.1; Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1) ;" at times. Notice both IE 6 and 7 mentioned. When it does this, the uploader popup is shifted to the bottom of the browser window and part of it is hidden.

I can reproduce this by setting the UA string in Firefox to "Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 5.1; Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1) ; .NET CLR 1.1.4322; .NET CLR 2.0.50727)" as an example.

Attached screenshot was taken using the Firefox method.

Attachments (2)

ie6-7-uastring.jpg (67.8 KB) - added by MtDewVirus 6 years ago.
TB-position.patch (2.2 KB) - added by azaozz 6 years ago.

Download all attachments as: .zip

Change History (7)

MtDewVirus6 years ago

comment:1 MtDewVirus6 years ago

  • Resolution set to fixed
  • Status changed from new to closed

Problem is resolved with other updates to Thickbox.

comment:2 MtDewVirus6 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Confirmed in r8190 that it's happening again.

comment:3 MtDewVirus6 years ago

  • Milestone changed from 2.5.2 to 2.6
  • Version changed from 2.5 to 2.6

azaozz6 years ago

comment:4 azaozz6 years ago

Browser detection fails if the browser pretends to be IE6. Testing for features instead.

comment:5 ryan6 years ago

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

(In [8194]) Test for feature instead of browser to fix TB positioning for browsers pretending to be IE6. Props azaozz. fixes #6676

Note: See TracTickets for help on using tickets.