Opened 14 years ago
Closed 14 years ago
#8321 closed Bug (fixed)
[Regression] Enter key doesn't keep inline styles
| Reported by: | Garry Yao | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.6.2 |
| Component: | UI : Enter Key | Version: | 3.6.2 |
| Keywords: | Regression IBM | Cc: | satya_minnekanti@… |
Description
- Load the editor with the following content and selection:
<p><strong>foo^</strong></p>
- Press enter key at the cursor position;
- Try type more text at the new paragraph.
- Actual: Cursor inserted texts are not in bold.
Attachments (3)
Change History (17)
comment:1 follow-up: 5 Changed 14 years ago by
| Keywords: | Firefox Webkit added |
|---|---|
| Status: | new → confirmed |
Changed 14 years ago by
| Attachment: | 8321.patch added |
|---|
comment:2 Changed 14 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | confirmed → review |
comment:3 Changed 14 years ago by
| Cc: | satya_minnekanti@… added |
|---|
comment:4 Changed 14 years ago by
| Status: | review → review_failed |
|---|
The following patch fails after patch:
http://ckeditor.t/dt/core/dom/element.html
comment:5 Changed 14 years ago by
| Keywords: | Firefox Webkit removed |
|---|
Replying to garry.yao:
Regression of [7242], broken dt: http://ckeditor.t/dt/plugins/enter/blockquote.html
Test broken on IE as well.
comment:6 Changed 14 years ago by
Regression failures moved out of ckeditor-tests master to t/8321
http://ckeditor.t/dt/plugins/enter/blockquote.html
comment:8 Changed 14 years ago by
| Keywords: | Regression added |
|---|
comment:9 Changed 14 years ago by
| Status: | review_failed → review |
|---|
Changed 14 years ago by
| Attachment: | 8321_2.patch added |
|---|
comment:10 Changed 14 years ago by
| Status: | review → review_failed |
|---|
I've updated the tests and these are still failing after patch:
http://ckeditor.t/dt/plugins/enter/blockquote.html
http://ckeditor.t/tt/8321/1.html
comment:11 Changed 14 years ago by
| Keywords: | IBM added |
|---|
Changed 14 years ago by
| Attachment: | 8321_3.patch added |
|---|
comment:12 Changed 14 years ago by
| Status: | review_failed → review |
|---|
comment:13 Changed 14 years ago by
| Status: | review → review_passed |
|---|
comment:14 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [7249].

Regression of [7242], broken dt: http://ckeditor.t/dt/plugins/enter/blockquote.html