-
-
Notifications
You must be signed in to change notification settings - Fork 60
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.
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
Make picture/srcset work #81
Comments
PR #83 should (partially?) fix this issue. Note that, in view mode, the rendered HTML is now the following (seen with developer tool): <picture>
<span data-ck-unsafe-element="source" srcset="https://interactive-examples.mdn.mozilla.net/media/cc0-images/surfer-240-200.jpg" media="(orientation: portrait)"></span>
<img src="https://interactive-examples.mdn.mozilla.net/media/cc0-images/painted-hand-298-332.jpg" alt="">
</picture> While it displays the image, it's still replacing |
It looks like removing Note that for dependency reasons, |
@marob Hi, you can easily remove plugins you don't want for your project via configuration of the plugin. Also, the same is for adding plugins. |
Hi @nshenderov. I found 2 ways:
Is there a better and easier way? Also, do you think adding |
I can't find how to make picture/srcset feature to work with that plugin.
If I paste that code after switching to source editing:
it's not displayed in view mode and it's replaced by the following code (discovered with developed tools):
I've checked that same example on Full-featured editor and that's working.
I've tried overriding config with the following one (thinking maybe
disallow
orsanitizeHtml
were the culprits), but that's still not working:I've even tried upgrading to 37.0.0-alpha.0 version of CKEditor as I've found some work was done about that feature in that version, but still no success.
I suspect maybe ckeditor5-easy-image plugin is missing, but I have not bean able to test adding it as I can't find how to do that without having to create a fork of your plugin.
The text was updated successfully, but these errors were encountered: