Opened 14 years ago
Closed 13 years ago
#7912 closed Bug (fixed)
Enter key does not work properly
| Reported by: | Lynne Kues | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.6.2 |
| Component: | UI : Enter Key | Version: | 3.0 |
| Keywords: | IBM | Cc: | satya_minnekanti@… |
Description (last modified by )
IE8
- Load this HTML:
<p>test test<span style="display: none"> </span></p> <p>and this is some more text</p> <p>test<span style="display: none"> </span></p> <p> </p> <p> </p>
- Put cursor at end of third line of text by clicking the mouse.
- Press Enter key.
Nothing happens. The cursor does not go to the next line.
Attachments (2)
Change History (18)
comment:1 Changed 14 years ago by
| Status: | new → confirmed |
|---|---|
| Summary: | IE - Enter key does not work properly → Enter key does not work properly |
| Version: | 3.4.2 → 3.0 |
comment:2 Changed 14 years ago by
| Cc: | satya_minnekanti@… added |
|---|
comment:3 Changed 14 years ago by
This is very confusing behavior for our customers. It's not apparent that new lines are being added and it appears that you can't add text to the end of the document.
comment:4 Changed 14 years ago by
It also affects arrow keys. E.g. in the above scenario try pressing the left arrow key instead of enter - we have to actually press it 5 times in order to move the caret one character to the left. And then try to go to the next line using the right arrow key :)
Looks like IE traverses through invisible content (diplay:none) just like through regular one. We can even freely modify this content - if we are "inside of" an invisible element then we can type anything and it'll be changed (we can verify this behavior by switching later to the source view).
A solution for this problem would probably require adjusting caret position, so that it will never be inside of invisible element.
Changed 13 years ago by
| Attachment: | 7912.patch added |
|---|
comment:5 Changed 13 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | confirmed → review |
comment:7 Changed 13 years ago by
| Description: | modified (diff) |
|---|
comment:8 Changed 13 years ago by
comment:9 Changed 13 years ago by
| Status: | review → review_failed |
|---|
The patch is making no difference to me.
Changed 13 years ago by
| Attachment: | 7912_2.patch added |
|---|
comment:10 Changed 13 years ago by
| Component: | General → UI : Enter Key |
|---|---|
| Status: | review_failed → review |
comment:11 Changed 13 years ago by
Unfortunately due to an selection system bug in Opera, which prevents us from making the following selection, we're not able to resolve the issue in that browser, new ticket would be opened to track it.
// The following selection is never possible in Opera, browser will redirect cursor to the paragraph above it. <p>^<span style="display:none">text</span></p>
comment:12 follow-up: 14 Changed 13 years ago by
| Status: | review → review_failed |
|---|
We're almost there, but the test is still failing with IE9/Compat. http://ckeditor.t/tt/7912/1.html
It passes with IE9 mode and IE8.
comment:13 Changed 13 years ago by
Additionally, this one fails with FF4: http://ckeditor.t/tt/4781/1.html
comment:14 Changed 13 years ago by
| Status: | review_failed → review |
|---|
Replying to fredck:
but the test is still failing with IE9/Compat.
IE < 8 doesn't really hide display:none elements thus are free of this bug, excluded from TC now.
Additionally, this one fails with FF4: http://ckeditor.t/tt/4781/1.html
It should be caused by outdated ticket branch, WFM after merged from master.
comment:15 Changed 13 years ago by
| Milestone: | → CKEditor 3.6.2 |
|---|---|
| Status: | review → review_passed |
comment:16 Changed 13 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [7242].
This only affects wysiwyg mode but when you switch to source everything is as it should be.
On IE8 and IE9 new paragraphs are added but the cursor doesn't move.
On FF new paragraphs are added but the cursor doesn't move or disappears.
On Webkit if you place the cursor at the end of first line (behind test test) and press enter, cursor will move to second line although new paragraph was made (see the source)
On Opera pressing enter ,when placing cursor behind third or first line, moves the cursor to the beginning of the text.
This has been true from CKEditor 3.0