Opened 14 years ago
Closed 13 years ago
#7955 closed Bug (fixed)
FF: with Auto grow plugin(with it's default configuration) PageUp & PageDown not working
Reported by: | Satya Minnekanti | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.6.3 |
Component: | Core : Selection | Version: | 3.4 |
Keywords: | IBM Firefox | Cc: | Damian, James Cunningham, Teresa Monahan |
Description
To reproduce the defect:
- Open AutoGrow sample and enter content that spans multiple pages.
- Press PageUp button.
Expected Result: Cursor in editor body moves one page up.
Actual Result: Cursor goes out of editor body and we can't type in the editor.
Attachments (1)
Change History (14)
comment:1 Changed 14 years ago by
Summary: | FF: with Auto grow plugin PageUp & PageDown not working → FF: with Auto grow plugin(with it's default configuration) PageUp & PageDown not working |
---|
comment:2 Changed 14 years ago by
Status: | new → confirmed |
---|---|
Summary: | FF: with Auto grow plugin(with it's default configuration) PageUp & PageDown not working → with Auto grow plugin(with it's default configuration) PageUp & PageDown not working |
Version: | → 3.4 |
comment:4 Changed 14 years ago by
Summary: | with Auto grow plugin(with it's default configuration) PageUp & PageDown not working → FF: with Auto grow plugin(with it's default configuration) PageUp & PageDown not working |
---|
comment:5 Changed 14 years ago by
we will get the same error even when we set maxHeight to large vale(for ex: 5000px) and when there are no scroll bars for the entered content
comment:7 Changed 13 years ago by
This is still reproducible in the nightly builds. The issue in FF is particularly bad because it is not possible to type in the editor at all after the Page Up button is pressed. The cursor seems to get lost and users need to tab to the editor again or mouse click into it, to be able to enter text again.
comment:8 Changed 13 years ago by
This is also an issue in FF when not using autogrow.
To reproduce:
- Open the replacebyclass sample.
- Enter a couple of lines of text. Note: Make sure the editor scroll bar does not appear.
- Press 'Page Up'
Problem: The cursor is now lost. It is not possible to enter text into the editor.
If you enter enough text to display the editor scroll bar, this issue does not occur.
An interesting thing to note is that when 'Page Up' is clicked, the Format drop down changes from 'Normal' to 'Heading 1'.
comment:10 Changed 13 years ago by
Replying to garry.yao:
The above tc WFM in latest FF.
Hi Garry, This testcase only works if the browser scroll bar is present. If you re-size your browser window so that the browser scroll bar appears and then re-try the steps above, you should be able to reproduce the issue. We actually believe this is a FF issue and we have logged a bug for it at https://bugzilla.mozilla.org/show_bug.cgi?id=695660. It would be good if you could comment on this ticket to try to escalate it's priority.
Is there any way CKEditor can workaround this issue until FF gets a fix?
Changed 13 years ago by
Attachment: | 7955.patch added |
---|
comment:11 Changed 13 years ago by
Component: | General → Core : Selection |
---|---|
Keywords: | Firefox added |
Owner: | set to Garry Yao |
Status: | confirmed → review |
The bug affects only when editor doc is not scrolled and page doc is scrolled, and I see a workaround within the editor is still feasible.
comment:12 Changed 13 years ago by
Milestone: | → CKEditor 3.6.3 |
---|---|
Status: | review → review_passed |
Interesting solution.
Hopefully we'll be able to remove this hack from the code once the original Firefox bug gets fixed.
comment:13 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [7369].
True for Firefox browsers from CKEditor 3.4. FF works fine when autoGrow_maxHeight is set but with default configuration the “Cursor goes out of editor body and we can't type in the editor”.
Few notes about IE (and Opera)
IE8, IE9 and Opera - with autoGrow_maxHeight there are no "pages", pressing page up moves cursor to the beginning of the text (no big deal). With default configuration pressing page-up moves the cursor to the beginning of the text but the browser doesn't scroll (Something like the first part of my comment in ticket #7946).
IE7 - with autoGrow_maxHeight set only the browser is scrolled. Default configuration works here.
IE6 - with autoGrow_maxHeight set scrolling works fine. With default configuration pressing page-up moves the cursor to the beginning of the text but the browser doesn't scroll just like IE9, IE8 and Opera.
In Webkit scrolling works.