Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#17329 closed defect (bug) (fixed)

Inline media uploader broken

Reported by: greuben's profile greuben Owned by: azaozz's profile azaozz
Milestone: 3.2 Priority: normal
Severity: major Version:
Component: General Keywords: needs-testing
Focuses: Cc:

Description

Using FF4, on add new posts screen click on "Add an image" meida button, the "select files" button is missing and there's a javascript error.

c.defaultView.getComputedStyle(a, null) is null

The button appears if I reload the media uploader iframe

Attachments (2)

17329.diff (863 bytes) - added by greuben 14 years ago.
17329.2.diff (3.2 KB) - added by greuben 14 years ago.

Download all attachments as: .zip

Change History (13)

#2 follow-up: @azaozz
14 years ago

If we init the JS later, perhaps at body.onload instead of document.ready, would it bypass that bug?

@greuben
14 years ago

#3 in reply to: ↑ 2 @greuben
14 years ago

Replying to azaozz:

If we init the JS later, perhaps at body.onload instead of document.ready, would it bypass that bug?

Tried it, no change.

A silly hack to bypass this bug would be changing display: none to visibility: hidden. See 17329.diff

#4 @nacin
14 years ago

  • Milestone changed from Awaiting Review to 3.2

#5 @greuben
14 years ago

The js error is fixed in jQuery 1.6.1rc1 but the "Select files" button is still not displayed with jQuery 1.6.1rc1

Last edited 14 years ago by greuben (previous) (diff)

#6 follow-up: @azaozz
14 years ago

Seems fixed in FF 4.0.1, all thickbox popups with iframes work properly even with jQuery 1.5.2.

Last edited 14 years ago by azaozz (previous) (diff)

#7 in reply to: ↑ 6 @greuben
14 years ago

Replying to azaozz:

Seems fixed in FF 4.0.1, all thickbox popups with iframes work properly even with jQuery 1.5.2.

I'm using FF 4.0.1(Ubuntu), the problem still exists.

#8 @johnbillion
14 years ago

Confirming this error occurs in Firefox 4.0 on Win7.

@greuben
14 years ago

#9 @greuben
14 years ago

  • Keywords needs-testing added

A little less creepy hack is to use negative offsets instead of display: none. See 17329.2.diff

#10 @azaozz
14 years ago

Think the reason for this happening is that the swfobject DOM ready event fires before the jQuery one. Tried adding some delay in swfupload.swfobject.js that seems to fix this.

#11 @azaozz
14 years ago

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

In [18073]:

Add some delay to allow the flash uploader swf to load in FF4, fixes #17329

Note: See TracTickets for help on using tickets.