Opened 7 years ago
Closed 7 years ago
#4397 closed Bug (fixed)
Ambiguous page scroll
| Reported by: | tobiasz.cudnik | Owned by: | garry.yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.1 |
| Component: | General | Version: | |
| Keywords: | Firefox Confirmed Review+ | Cc: | batch2k@…, lens@… |
Description
Ticket migrated from forum thread.
I have several CKEditor windows on a single page; they are invoked by a simple "ckeditor" style selector. Unexpectedly, after all plugin instances get initialized, page spontaneously scrolls to the lowest editor window (not smooth scrolling but a single-jump).
Reported to be FF only issue.
Attachments (2)
Change History (14)
comment:1 Changed 7 years ago by batch2k
- Cc batch2k@… added
comment:2 Changed 7 years ago by lens
- Cc lens@… added
comment:3 Changed 7 years ago by garry.yao
- Keywords Confirmed added
- Milestone set to CKEditor 3.1
- Owner set to garry.yao
- Status changed from new to assigned
Changed 7 years ago by garry.yao
comment:4 Changed 7 years ago by garry.yao
- Keywords Review? added
The design mode/contenteditable in FF is the culprit which cause host page scrolling.
comment:5 Changed 7 years ago by tobiasz.cudnik
- Keywords Review+ added; Review? removed
Firefox 3.5 is not affected (at least linux version).
comment:6 Changed 7 years ago by garry.yao
- Resolution set to fixed
- Status changed from assigned to closed
I think all linux version is not affected ;).
Fixed with [4219].
comment:7 Changed 7 years ago by batch2k
I downloaded updated file (plugins/wysiwygarea/plugin.js, zip-file) from the ticket page. Well, this seems not working as well :-( Maybe I did something wrong and chosen incorrect file or something?..
Windows/FF 3.5
comment:8 Changed 7 years ago by alfonsoml
#4372 has been marked as dup
comment:9 Changed 7 years ago by garry.yao
- Keywords Review+ removed
- Resolution fixed deleted
- Status changed from closed to reopened
The previous diagnose is wrong because it's actually a regression of [3816], where the key simulation cause the page to scroll, that hack should actually been moved somewhere else.
Changed 7 years ago by garry.yao
comment:10 Changed 7 years ago by garry.yao
- Keywords Review? added
The committed changes will be reverted once this patch is accepted.
comment:11 Changed 7 years ago by fredck
- Keywords Review+ added; Review? removed
Please fix the code indentation before committing, and include the changelog entry for it.
comment:12 Changed 7 years ago by garry.yao
- Resolution set to fixed
- Status changed from reopened to closed
Fixed with [4295].
Ticket Test added at : http://ckeditor.t/tt/4397/1.html.