Opened 10 years ago

Closed 10 years ago

#4344 closed Bug (fixed)

Successive <br>s get lost

Reported by: Frederico Caldeira Knabben Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.1
Component: General Version: 3.0
Keywords: Confirmed Review+ Cc: nishimoto.s@…

Description

  1. Load the following HTML:
<p>AAA<br />
<br /></p>
<p>XXX</p>
  1. Move to wysiwyg view. Note that there is no extra line space between AAA and BBB. The BRs got already lost.
  1. Move to source. You have the following:
<p>
	AAA</p>
<p>
	XXX</p>

Expected:

<p>
	AAA<br>
	&nbsp;</p>
<p>
	XXX</p>

Attachments (1)

4344.patch (2.4 KB) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 10 years ago by Garry Yao

Owner: set to Garry Yao
Status: newassigned

Changed 10 years ago by Garry Yao

Attachment: 4344.patch added

comment:2 Changed 10 years ago by Garry Yao

Keywords: Review? added

comment:3 Changed 10 years ago by Garry Yao

The culprit is about white-spaces between filler nodes.

comment:4 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

When committing:

  • The test case must be moved to the proper place now.
The "
null" is not needed when returning from lastNoneSpaceChild. You may remove it.
  • Changelog.

comment:5 Changed 10 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy