Opened 13 years ago
Closed 13 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:
- Open any CKEditor sample and switch to Source mode.
- Enter the following:
<ul> This is a sample that contains <b>bold</b> text </ul>
- 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)
Change History (5)
comment:1 Changed 13 years ago by
Status: | new → confirmed |
---|---|
Version: | 3.6.3 (SVN - trunk) → 3.0 |
Changed 13 years ago by
Attachment: | 8540.patch added |
---|
comment:2 Changed 13 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | confirmed → review |
comment:3 Changed 13 years ago by
Component: | General → Core : Parser |
---|---|
Milestone: | → CKEditor 3.6.3 |
Status: | review → 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 13 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [7411].
Reproducible in all browsers from CKEditor 3.0