Opened 15 years ago
Closed 15 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 15 years ago by
| Status: | new → confirmed | 
|---|
comment:2 Changed 15 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 15 years ago by
| Description: | modified (diff) | 
|---|
Changed 15 years ago by
| Attachment: | 7070.patch added | 
|---|
comment:4 Changed 15 years ago by
| Owner: | set to Sa'ar Zac Elias | 
|---|---|
| Status: | confirmed → review | 
comment:5 Changed 15 years ago by
| Status: | review → review_failed | 
|---|
Second TC with a table is still not working.
comment:6 Changed 15 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 15 years ago by
| Owner: | changed from Sa'ar Zac Elias to Garry Yao | 
|---|---|
| Status: | review_failed → assigned | 
Changed 15 years ago by
| Attachment: | 7070_2.patch added | 
|---|
comment:8 Changed 15 years ago by
| Owner: | changed from Garry Yao to Sa'ar Zac Elias | 
|---|---|
| Status: | assigned → review | 
comment:9 Changed 15 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 15 years ago by
| Owner: | changed from Sa'ar Zac Elias to Garry Yao | 
|---|---|
| Status: | review_failed → assigned | 
Changed 15 years ago by
| Attachment: | 7070_3.patch added | 
|---|
comment:11 Changed 15 years ago by
| Status: | assigned → review | 
|---|
Ticket Test added: 
run  OR view source.
comment:12 Changed 15 years ago by
| Status: | review → review_passed | 
|---|
comment:13 Changed 15 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.