Ticket #8540 (closed Bug: fixed)

Opened 2 years ago

Last modified 2 years ago

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

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

Change History

comment:1 Changed 2 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 2 years ago by garry.yao

comment:2 Changed 2 years ago by garry.yao

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

comment:3 Changed 2 years ago by fredck

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

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 2 years ago by garry.yao

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

Fixed with [7411].

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