Opened 15 years ago
Closed 15 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 15 years ago by
| Cc: | batch2k@… added |
|---|
comment:2 Changed 15 years ago by
| Cc: | lens@… added |
|---|
comment:3 Changed 15 years ago by
| Keywords: | Confirmed added |
|---|---|
| Milestone: | → CKEditor 3.1 |
| Owner: | set to Garry Yao |
| Status: | new → assigned |
Changed 15 years ago by
| Attachment: | 4397.patch added |
|---|
comment:4 Changed 15 years ago by
| Keywords: | Review? added |
|---|
The design mode/contenteditable in FF is the culprit which cause host page scrolling.
comment:5 Changed 15 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
Firefox 3.5 is not affected (at least linux version).
comment:6 Changed 15 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
I think all linux version is not affected ;).
Fixed with [4219].
comment:7 Changed 15 years ago by
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:9 Changed 15 years ago by
| Keywords: | Review+ removed |
|---|---|
| Resolution: | fixed |
| Status: | closed → 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 15 years ago by
| Attachment: | 4397_2.patch added |
|---|
comment:10 Changed 15 years ago by
| Keywords: | Review? added |
|---|
The committed changes will be reverted once this patch is accepted.
comment:11 Changed 15 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
Please fix the code indentation before committing, and include the changelog entry for it.
Ticket Test added at : http://ckeditor.t/tt/4397/1.html.