Opened 4 years ago

Closed 3 years ago

#8540 closed Bug (fixed)

Styling lost on text that is included within an invalid HTML block

Reported by: tmonahan Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.6.3
Component: Core : Parser Version: 3.0
Keywords: IBM Cc: damo, satya

Description

Steps to Reproduce:

  1. Open any CKEditor sample and switch to Source mode.
  2. Enter the following:
    <ul>
    This is a sample that contains <b>bold</b> text
    </ul>
    
  3. Switch back to wysiwyg mode.

Problem: The bold styling is not displayed.

If you switch back to Source mode without clicking in the editor body, you can see the source has changed to:

<ul>
	This is a sample that contains bold text
</ul>

The bold formatting is already lost.

If you switch back to wysiwyg mode, click in the editor and then switch back to Source mode, the source is now:

<p>
	This is a sample that contains bold text</p>
<ul>
</ul>

We realise that using the UL tag without an inner LI is not valid HTML. However we would still expect the styling of the text content to be preserved.

Attachments (1)

8540.patch (956 bytes) - added by garry.yao 3 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 4 years ago by j.swiderski

  • Status changed from new to confirmed
  • Version changed from 3.6.3 (SVN - trunk) to 3.0

Reproducible in all browsers from CKEditor 3.0

Changed 3 years ago by garry.yao

comment:2 Changed 3 years ago by garry.yao

  • Owner set to garry.yao
  • Status changed from confirmed to review

comment:3 Changed 3 years ago by fredck

  • Component changed from General to Core : Parser
  • Milestone set to CKEditor 3.6.3
  • Status changed from review to review_passed

I see this far from being a priority issue, but it is a welcomed enhancement.

Btw, I was not able to find the above test, but I've noticed enhancements to the following instead:
http://ckeditor.t/dt/core/htmlparser/fragment.html

comment:4 Changed 3 years ago by garry.yao

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

Fixed with [7411].

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