- πΊπΈUnited States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request β as a guide.
@gaurav-mathur @Akhildev.cs appears screenshots were already in #34 so these were duplicated efforts unfortunately.
Hiding all the screenshot noise
Appears this was tagged for tests also which still need to happen.
I did not do a code review.
- πΊπΈUnited States loze Los Angeles
What version of ckeditor is this supposed to work with? I do not see any edit button after applying the patch using ckedtor5
- π³πΏNew Zealand ericgsmith
@loze that patch will not work with CKEditor5 as CKEditor5 does not load the form that it is altering.
I have added an MR to the edit_media_modal module with an approach for CKEditor 5 https://www.drupal.org/project/edit_media_modal/issues/3352274 β¨ Support CKEditor5 Needs review