Opened 10 years ago

Closed 10 years ago

#8540 closed Bug (fixed)

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

Reported by: Teresa Monahan Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.6.3
Component: Core : Parser Version: 3.0
Keywords: IBM Cc: Damian, Satya Minnekanti

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 10 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 10 years ago by Jakub Ś

Status: newconfirmed
Version: 3.6.3 (SVN - trunk)3.0

Reproducible in all browsers from CKEditor 3.0

Changed 10 years ago by Garry Yao

Attachment: 8540.patch added

comment:2 Changed 10 years ago by Garry Yao

Owner: set to Garry Yao
Status: confirmedreview

comment:3 Changed 10 years ago by Frederico Caldeira Knabben

Component: GeneralCore : Parser
Milestone: CKEditor 3.6.3
Status: reviewreview_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 10 years ago by Garry Yao

Resolution: fixed
Status: review_passedclosed

Fixed with [7411].

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