Opened 13 years ago

Closed 12 years ago

#1967 closed defect (fixed)

scrolling while Dialog shown doesn't work on FF

Reported by: bill Owned by: bill
Priority: high Milestone: 0.9beta
Component: Dijit Version: 0.4.1rc1
Keywords: Cc:
Blocked By: Blocking:

Description

On IE, if you show a Dialog and then scroll, the background will be adjusted, but on FF it isn't. (Just try test_Dialog.html)

Attachments (1)

dialog.patch (919 bytes) - added by Jonathan Bond-Caron 13 years ago.
Connect to document element instead of window

Download all attachments as: .zip

Change History (8)

comment:1 Changed 13 years ago by bill

Milestone: 0.4.10.6

Changed 13 years ago by Jonathan Bond-Caron

Attachment: dialog.patch added

Connect to document element instead of window

comment:2 Changed 13 years ago by Jonathan Bond-Caron

Small typo, the disconnect should actually be:

dojo.event.disconnect(document.documentElement, "onscroll", this, "_onScroll");

The window onscroll support for firefox seems to be buggy,

tested on firefox 1.5.0.8, IE 6.0, Opera 8

jon

comment:3 Changed 13 years ago by Thorsten Berger

We ran into this problem too. Thanks for the patch. Unfortunately the scrolling is now broken on Firefox 2.0. The workaround is to both connect/disconnect to the element and the window.

Regards, Thorsten

comment:4 Changed 13 years ago by Jonathan Bond-Caron

Just for the record, this is related to:

http://trac.dojotoolkit.org/ticket/1406

Using the following: document.documentElement.addEventListener("onscroll", block, false);

Will cause the event not to bubble up to the window This is a bug in Firefox <= 2.0 But apparently, this has been fixed in the latest firefox build.

I'll submit a proposed patch to fix this issue (through dojo.event.connect)

jon

comment:5 Changed 12 years ago by bill

Component: WidgetsDijit
Owner: changed from bill to koranteng

comment:6 Changed 12 years ago by bill

Milestone: 1.00.9beta
Owner: changed from koranteng to bill
Status: newassigned

comment:7 Changed 12 years ago by bill

Resolution: fixed
Status: assignedclosed

(In [9082]) Fix various dialog bugs, and put opacity in CSS file, rather than JS file. Fixes #3274, #3393, #1967. Still some problems w/href but that is another bug (refs #3373).

Note: See TracTickets for help on using tickets.