Opened 15 years ago
Closed 14 years ago
#3721 closed defect (fixed)
0.9: base,html.rtl tests fail on IE 6
Reported by: | Jared Jurkiewicz | Owned by: | Adam Peller |
---|---|---|---|
Priority: | high | Milestone: | 1.1 |
Component: | HTML | Version: | 0.9 |
Keywords: | bidi | Cc: | [email protected]… |
Blocked By: | Blocking: |
Description
PASSED test: ../../dojo/tests/_base/html_rtl.html::t::coordsWithHorzScrollbar
[object Error] ERROR IN: function eventClientXY(t){ IE only test if(dojo.isIE){ show vertical scrollbar dojo.byId("rect_vert").style.display = ""; var rect = dojo.byId("rect100"); var assertException = null; function rect_onclick(e){ move the rectangle to the mouse point rect.style.left = e.pageX + "px"; rect.style.top = e.pageY + "px"; window.alert("Do NOT move your mouse!!!
" + "The black rectangle's top-left point should be under the mouse point.
" + "If not, you will see a failure in the test report later.
" + "Now press the space bar, but do NOT move your mouse."); rect.fireEvent('ondblclick'); } function rect_ondblclick(){ test if the rectangle is really under the mouse point try{ t.is(0, event.offsetX); t.is(0, event.offsetY); }catch (e){ allow the exception in a event handler go to the event firer assertException = e; } } dojo.connect(rect, "onclick", null, rect_onclick); dojo.connect(rect, "ondblclick", null, rect_ondblclick); window.alert("Move the mouse to anywhere in this page, and then press the space bar."); rect.fireEvent('onclick'); if(assertException != null){ throw assertException; } } }
FAILED test: ../../dojo/tests/_base/html_rtl.html::t::eventClientXY PASSED test: ../../dojo/tests/_base/html_rtl.html
Attachments (1)
Change History (8)
comment:1 Changed 15 years ago by
Owner: | changed from Bryan Forbes to Adam Peller |
---|
comment:2 Changed 15 years ago by
Resolution: | → invalid |
---|---|
Status: | new → closed |
seems to work after retesting
comment:3 Changed 15 years ago by
Keywords: | bidi added |
---|
comment:4 Changed 15 years ago by
Milestone: | 0.9 → 1.1 |
---|---|
Resolution: | invalid |
Status: | closed → reopened |
this appears when you runTests.html in ie6 or 7 for me, and was reported today on #dojo
Changed 15 years ago by
Attachment: | TestcaseDojoCoordsBugIE.html added |
---|
comment:5 Changed 15 years ago by
This Bug is caused by the dojo._fixIeBiDiScrollLeft function. The clientWidth is always read by the dojo.doc.documentElement. This returns 0 in IE >=6 if the browser is in Quirksmode. In Quirksmode the body-Element needs to be used. Replacing the de-variable in dojo._fixIeBiDiScrollLeft with following fixes the problem:
var de = ((dojo.doc["compatMode"] == "BackCompat") || (dojo.isIE < 6)) ? dojo.doc.body : dojo.doc.documentElement;
Regards, makin
comment:6 Changed 14 years ago by
Cc: | [email protected]… added |
---|
I still see a UT failure with this change.
hmm... worked for me