Opened 14 years ago
Last modified 13 years ago
#7080 confirmed Bug
clear selection cause window scroll
Reported by: | yiminghe | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | Core : Selection | Version: | 3.2.1 |
Keywords: | IE6 IE7 | Cc: |
Description
happen in ie6
steps: 1.place a button far below editor :
<textarea id='test'></textarea> <div style='height:1500px;'></div> <button>click</button> <script>CKEDITOR.replace("test");</script>
2.insert a page-break('插入分页符') into editor
3.keep focus(cursor) in the editor and scroll to 'click button' using mouse wheel
4.click button
expected :
window does not scroll
actual :
window scrolls to editor
reason :
caused by line 184 in _source/plugins/selection/plugin.js
doc.selection.empty() causes scrolling.
we should record scrollTop and scrollLeft before empty() and restore them after empty() , i provide a patch ,hope helpful.
Attachments (2)
Change History (8)
Changed 14 years ago by
Attachment: | scroll.patch added |
---|
Changed 14 years ago by
Attachment: | 7080_TC.html added |
---|
comment:1 Changed 14 years ago by
Keywords: | scroll selection empty removed |
---|---|
Status: | new → pending |
comment:2 Changed 14 years ago by
yes , i can reproduce on 7080_TC.html:
steps :
- insert a page-break('插入分页符')(or image,smiley) into editor (important!).
- keep focus(cursor) in the editor (important) and scroll to 'click button' using mouse wheel (important) .
- click button
expected :
window does not scroll
actual :
window scrolls to editor
ps :see screen video attached
comment:4 Changed 13 years ago by
Keywords: | IE6 IE7 added |
---|---|
Status: | pending → confirmed |
Version: | 3.5.1 → 3.2.1 |
This issue has reproducible in IE6 and IE7 from CKEditor 3.2.1.
- Copy attached TC to samples folder
- Click inside editor
- Insert page break by clicking on a toolbar button
- Scroll down to button and click on it
Result: Browser will scroll up to cursor location.
comment:6 Changed 13 years ago by
Status: | pending → confirmed |
---|
I have just checked TC from comment:4.
Problem is still reproducible on latest trunk.
WFM with attached TC, any clue?