Opened 7 years ago

Closed 6 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

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 changed from new to assigned

comment:3 Changed 6 years ago by fredck

  • Milestone changed from CKEditor 3.4 to CKEditor 3.5

comment:4 Changed 6 years ago by garry.yao

  • Milestone changed from CKEditor 3.4.2 to CKEditor 3.6

Not of unimportance, defer it

comment:5 Changed 6 years ago by fredck

  • Milestone changed from CKEditor 3.6 to CKEditor 3.5.4
  • Status changed from review to review_passed

Minor fix. Re-targeting.

comment:6 Changed 6 years ago by garry.yao

  • Resolution set to fixed
  • Status changed from review_passed to closed

Fixed with [6740].

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