Opened 6 years ago

Closed 6 years ago

#11049 closed Bug (wontfix)

Wrong validation for style tag if a quote for the type attribute is missing

Reported by: ePagesTeam Owned by:
Priority: Normal Milestone:
Component: General Version: 4.1.2
Keywords: Cc:

Description

When inserting the following code to the source mode

<style type=text/css"></style>

than switching to the WYSIWYG mode and back to source mode the editor generates following code:

<style type="text/css&quot;"></p></body></html>
</style>

This leads to invalid HTML.

When inserting other HTML tags, e.g. a H2 with missing quote, the editor validates this and adds the missing quote. We would expect the same behaviour here.

Change History (1)

comment:1 Changed 6 years ago by Jakub Ś

Resolution: wontfix
Status: newclosed

You are right, code you have presented causes such problem but why in the world would you insert such code into editor? Is there any real use case here?

If you insert quotes there is no problem and that is, I assume, in 99,9% of cases what users are doing.

On one hand yes the code looks weird but on the other we can't foresee and fix every possible case when user inserts invalid HTML. IMHO this is rare case and not critical thus I don't think we will work on this issue.

Proper solution here is using valid HTML. I'm closing this as "won't fix".

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