Opened 14 years ago
Closed 14 years ago
#7070 closed Bug (fixed)
Extra line break added to the list item when applying a style
Reported by: | Wiktor Walc | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.5.1 |
Component: | General | Version: | 3.5.1 |
Keywords: | Firefox | Cc: |
Description (last modified by )
Use the following selection:
<ul> <li> aaa [bbb</li> <li> ccc] ddd</li> </ul>
and apply bold, italic, underline, font color etc.
Result:
<ul> <li> aaa <u><em><strong>[bbb</strong></em></u><br /> <br /> <br /> </li> <li> <u><em><strong>ccc]</strong></em></u> ddd</li> </ul>
Caused by [6373].
Attachments (3)
Change History (16)
comment:1 Changed 14 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 14 years ago by
Keywords: | Firefox added |
---|---|
Summary: | [enterBr] Extra line break added to the list item when applying a style → Extra line break added to the list item when applying a style |
Note: it actually happens in all enter modes.
comment:3 Changed 14 years ago by
Description: | modified (diff) |
---|
Changed 14 years ago by
Attachment: | 7070.patch added |
---|
comment:4 Changed 14 years ago by
Owner: | set to Sa'ar Zac Elias |
---|---|
Status: | confirmed → review |
comment:5 Changed 14 years ago by
Status: | review → review_failed |
---|
Second TC with a table is still not working.
comment:6 Changed 14 years ago by
Second TC works for me. But there is a different problem with this patch (so R- anyway):
- Clear all contents
- Insert a table
- type some text in each cell
- Select All
- Apply Bold
- click belowa a table
- result: it is impossible to move below the table.
comment:7 Changed 14 years ago by
Owner: | changed from Sa'ar Zac Elias to Garry Yao |
---|---|
Status: | review_failed → assigned |
Changed 14 years ago by
Attachment: | 7070_2.patch added |
---|
comment:8 Changed 14 years ago by
Owner: | changed from Garry Yao to Sa'ar Zac Elias |
---|---|
Status: | assigned → review |
comment:9 Changed 14 years ago by
Status: | review → review_failed |
---|
Load the following content and selection and apply any inline style:
<p>[some <br/> <br/>] text</p> <p>another</p>
Actual Result: Lose one line-break;
comment:10 Changed 14 years ago by
Owner: | changed from Sa'ar Zac Elias to Garry Yao |
---|---|
Status: | review_failed → assigned |
Changed 14 years ago by
Attachment: | 7070_3.patch added |
---|
comment:11 Changed 14 years ago by
Status: | assigned → review |
---|
Ticket Test added:
run OR view source.
comment:12 Changed 14 years ago by
Status: | review → review_passed |
---|
comment:13 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [6382].
Another TC, enter mode = BR, paste the following in source:
select all, apply styles: bold, italic, underline. Result: extra line breaks are added to elements in the first row.