Opened 16 years ago
Closed 15 years ago
#3807 closed Bug (fixed)
[IE]Block style broken when cursor on front of text
Reported by: | Garry Yao | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.1 |
Component: | Core : Styles | Version: | |
Keywords: | IBM Confirmed | Cc: | damian.chojna@… |
Description
Reproducing Procedures
- Open the replace by class example page in IE;
- Load eidtor with the following content and selection;
<p>paragraph</p> ^text
- Click on any block style, e.g. 'Numbered List';
- Actual Result:
<ol> <li> paragraph</li> </ol> ^text
- Actual Result:
Attachments (2)
Change History (12)
comment:1 Changed 16 years ago by
Keywords: | Confirmed added |
---|---|
Owner: | set to Garry Yao |
Priority: | Normal → High |
Status: | new → assigned |
Changed 16 years ago by
Attachment: | 3807.patch added |
---|
comment:2 Changed 16 years ago by
Keywords: | Review? added |
---|
comment:3 Changed 16 years ago by
Summary: | [IE]domiterator broken with enterMode = BR → [IE]Block style broken when cursor on front of text |
---|
Update ticket summary, it's not dedicate to br entermode anymore.
comment:4 Changed 16 years ago by
Priority: | High → Normal |
---|
comment:5 Changed 16 years ago by
Keywords: | Review- added; Review? removed |
---|
- We should not throw the error in the selection change check function. There could be unknown selection issues, and we must avoid them breaking the editor code at that point, considering that this function is executed hundreds of times during the editor life cycle.
- The checkSelectionChange looks like the wrong place for this fix. The problem is with the selection class itself, within the getRanges function probably.
- The changelog is missing.
Changed 16 years ago by
Attachment: | 3807_2.patch added |
---|
comment:6 Changed 16 years ago by
Keywords: | Review? added; Review- removed |
---|
comment:7 Changed 16 years ago by
Cc: | damian.chojna@… added |
---|---|
Keywords: | IBM added |
comment:8 Changed 15 years ago by
Milestone: | CKEditor 3.0 → CKEditor 3.1 |
---|
comment:9 Changed 15 years ago by
Keywords: | Pending added |
---|
I could not reproduce this issue on newest trunk using on any IE. After trying revision [3772] i was able to get JS error in IE 6 and 7 (although different then described in ticket description).
Maybe this fix is not necessary due to other changes ?
comment:10 Changed 15 years ago by
Keywords: | Review? Pending removed |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
Unable to reproduce this bug anymore on trunk.
Actually I has wrong the domiterator from the title, it's an implicit bug of IE selection system, because when we make the selection as described above, you can figure out from the elementpath that the selection is instead inside above paragraph.
We need a fix for the selection system.