Opened 7 years ago

Closed 7 years ago

#5788 closed Bug (fixed)

HTML parser allows spaces after <br>

Reported by: Garry Yao Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.5.4
Component: Core : Output Data Version: SVN (CKEditor) - OLD
Keywords: Confirmed Cc:

Description

Whitespace in following HTML should be removed by HTML parser while it doesn't introduce any visual clue:

<p>test<br /> whitespace</p>

It could be verified from Firebug HTML tab's show whitespaces functionality.

#4574 ticket test is broken by this bug:
run OR view source.

Attachments (1)

5788.patch (546 bytes) - added by Garry Yao 7 years ago.

Download all attachments as: .zip

Change History (7)

Changed 7 years ago by Garry Yao

Attachment: 5788.patch added

comment:1 Changed 7 years ago by Garry Yao

Keywords: Review? added

comment:2 Changed 7 years ago by Garry Yao

Owner: set to Garry Yao
Status: newassigned

comment:3 Changed 7 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.4CKEditor 3.5

comment:4 Changed 7 years ago by Garry Yao

Milestone: CKEditor 3.4.2CKEditor 3.6

Not of unimportance, defer it

comment:5 Changed 7 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.6CKEditor 3.5.4
Status: reviewreview_passed

Minor fix. Re-targeting.

comment:6 Changed 7 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [6740].

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