Needs work
Project:
Video Filter
Version:
8.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
2 Sep 2022 at 15:37 UTC
Updated:
23 Feb 2026 at 12:09 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
damienmckennaComment #3
gaëlgI referenced this issue here: https://www.drupal.org/docs/core-modules-and-themes/core-modules/ckedito...
Comment #4
damienmckennaOfficial Drupal documentation for building a plugin:
https://www.drupal.org/docs/drupal-apis/ckeditor-5-api/overview
Official Drupal docs for rewriting a plugin:
https://www.drupal.org/docs/core-modules-and-themes/core-modules/ckedito...
Official CKEditor documentation for rewriting a plugin:
https://ckeditor.com/docs/ckeditor5/latest/installation/getting-started/...
Example module:
https://www.drupal.org/project/ckeditor5_dev
Blog posts on the topic:
https://evolvingweb.com/blog/porting-ckeditor-5-plugin-drupal-10
https://tech.axelerant.com/create-and-integrate-ckeditor-5-plugin-with-d...
Comment #5
damienmckennaA bare starting point - files from ckeditor5_dev copied & renamed to match.
One problem is that the ckeditor plugin doesn't want to work with shortcodes, so we might be stuck there.
Comment #6
damienmckennaMissed changes to one file.
Comment #7
damienmckennaThis may not be possible, I don't see any reference to using shortcodes in ckeditor5.
Comment #8
damienmckennaminnur created a merge request which provides a new approach for this functionality with ckeditor 5 and would be a great addition:
https://git.drupalcode.org/project/video_filter/-/merge_requests/7
While I think the better long-term approach would be to migrate to the Media module and just use 3rd party oembed integration, this could be a short-term solution for sites that aren't using the Media system.
Next steps
Split out the ckeditor5 changes from merge request #7 into its own merge request as much of the unrelated changes are being handled as separate issues.
Comment #9
damienmckennaAlso, now that there's test coverage (#3439443: Add useful functional test) we can finish this off to make sure it works as expected.
Comment #10
abu-zakham commentedThere is an unresolved conflict in MR7 that must be fixed.