Opened 11 years ago
Closed 11 years ago
#9859 closed defect (fixed)
scrollIntoView bug in Safari
Reported by: | haysmark | Owned by: | Douglas Hays |
---|---|---|---|
Priority: | high | Milestone: | 1.4 |
Component: | Dijit | Version: | 1.3.2 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
See attached test cases, which go in dijit/tests/_base.
In test_scroll_iframe, if you click "Scroll bold into view", the letter B in the iframe should scroll into view. In Safari 4, this fails. It seems that adding border or padding to the body of _insideOfIframe.html (as with the dijitTests css) causes this failure. Adding margin does not seem to do anything.
Attachments (4)
Change History (8)
Changed 11 years ago by
Attachment: | _insideOfIframe.html added |
---|
Changed 11 years ago by
Attachment: | 9859.patch added |
---|
Fixes #9859. Safari was adding the body's border and padding in scrollIntoView.
Changed 11 years ago by
Attachment: | 9859.2.patch added |
---|
Fixes #9859. Safari was adding the body's border and padding in scrollIntoView. Also fixed a bug with scrollIntoView with the robot and iframes.
comment:1 Changed 11 years ago by
Priority: | normal → high |
---|
comment:2 Changed 11 years ago by
Status: | new → assigned |
---|
comment:3 Changed 11 years ago by
(In [20428]) Fixes #10020. References #9859. Fixed several problems related to dojo.position()'s use inside iframes. Fixed several IE-specific problems related to inaccurate position() attributes caused by iframe style. Fixed withDoc to restore _isBodyLtr and to properly set/restore isQuirks. Changed isQuirks definition to only check for "BackCompat?" since other values are for ancient unsupported browsers. Added automated iframe tests to html.html. Removed workaround from dijit's scroll.js that's no longer needed after these fixes. Verified all html.html and test_scroll.html tests pass on all supported browsers. !strict
comment:4 Changed 11 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Supporting file. Goes in dijit/tests/_base