Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Img not closed #1876

Closed
BenoitChery opened this issue Jul 10, 2019 · 2 comments
Closed

Img not closed #1876

BenoitChery opened this issue Jul 10, 2019 · 2 comments
Labels
resolution:invalid This issue is invalid (e.g. reports a non-existent bug or a by-design behavior).

Comments

@BenoitChery
Copy link

BenoitChery commented Jul 10, 2019 •
edited
Loading

Is this a bug report or feature request? (choose one)

Other

💻 Version of CKEditor

ckeditor5

📋 Steps to reproduce

1.insert an image
2.the generated file contains not closed, isn't that mandatory in xhtml ?

✅ Expected result

❎ Actual result

## 📃 Other details that might be useful Can we configure this ?
@Mgsy
Copy link
Member

Mgsy commented Jul 11, 2019

Hello, indeed, tags should be closed in XHTML, but CKEditor 5 is based on HTML and the output generated by the editor is a valid HTML.

@Mgsy Mgsy closed this as completed Jul 11, 2019
@Mgsy Mgsy added the resolution:invalid This issue is invalid (e.g. reports a non-existent bug or a by-design behavior). label Jul 11, 2019
@ma2ciek
Copy link
Contributor

ma2ciek commented Jul 11, 2019 •
edited
Loading

Is the https://ckeditor.com/docs/ckeditor5/latest/api/module_engine_dataprocessor_xmldataprocessor-XmlDataProcessor.html what you're looking for?

And see the #1275 (comment) - you'll probably need to use the XMLSerializer too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
resolution:invalid This issue is invalid (e.g. reports a non-existent bug or a by-design behavior).
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ma2ciek @Mgsy @BenoitChery