Closed (fixed)
Project:
CKEditor Bootstrap Tabs
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
21 Sep 2022 at 11:34 UTC
Updated:
20 Nov 2023 at 16:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
ozinComment #4
emptyvoid commentedIt's not clear to me how I would generate a patch to the current release or integrate the two very different branches.
Comment #5
emptyvoid commentedI compared the two branches to 8.x-dev and the 3310866-ckeditor-5-support branch is identical to the current dev branch.
I then manually merged the ckeditor5-support branch into 8.x-dev.
Patch attached.
I'm attempting to test this using Ckeditor5 on Drupal 9.5.5.. we'll see if it loads.
Comment #6
emptyvoid commentedSo, this patch registers the icon into the editor configuration for a given input type in Drupal.
And when in edit mode for the body of a page it provide dialog inline in the editor to specify a tab set name and the number of tabs to include.
And that's when it fails.
It doesn't appear to load the templated structure into the DOM in the editor and simply paces two tab titles in a paragraph tag.
CSS fails, Javascript fails.
Though inspecting the code the bootstrapTabs plugin code does appear to be in place for legacy ckeditor4 as well as the new ckeditor5_plugins sub directory with code for a ck5 plugin. I'm not all that up on understanding the SDK and API integration between custom ckeditor5 plugins and the Drupal Plugin SDK and classes for integration.
Sure wish there was some articles and or example code for us to review.
So this is maybe 35% complete?
Comment #7
ozinHi @emptyvoid, thanks for your work.
We have a new dev branch 2.0.x, in this branch, we plan to implement CKEditor5.
Your patch is a good starting point, I hope development will move further faster now
Comment #8
s7434 commentedIs the dev branch 2.0.x inluded ckeditor5 support?
Comment #11
dylan donkersgoed commentedI've expanded this patch to handle upcasting/downcasting bootstrap tabs. You can now load an existing page with tabs on it, edit the tabs, and save it and have everything work (at least from some cursory testing).
There's still a bit of work to do (e.g. the button isn't working correctly to add/expand tables at the moment and you can't change the active tab) but I hope to have that done within the next couple days.
Comment #12
dylan donkersgoed commentedMR seems to be in a fully functional state now, attaching a patch.
There's probably room for some code cleanup, I'm not sure some of the separate upcast/downcast methods are still necessary, though they work fine.
Comment #13
dylan donkersgoed commentedComment #14
dylan donkersgoed commentedFew more fixes. Notably there was an error when trying to edit source with my last few changes and there was a conflict with the core list plugin.
Comment #15
dylan donkersgoed commentedForgot to actually build the plugin after the last change, new patch.
Comment #16
dylan donkersgoed commentedApparently I forgot to push so that was just the same patch as before. Here's the rebuilt patch for real.
Comment #17
ozinHi Dylan!
Thanks for the patch, I will take a look as soon as possible.
Comment #18
Jaspreet.gill commentedHi
I applied #16 patch but it is breaking HTML layout in ckeditor editor side.
Comment #19
robloachHi Mykhailo! Been using this on a site, and it has been working well :) ... Thanks so much for pushing this forwards, DylanDonkersgoed. Happy to help out with git pushes and the release if you need another co-maintainer.
Comment #20
robloach@DylanDonkersgoed I see your latest changes! Nice work. Could you add a basic composer.json in there? Something like this could be sufficient...
Comment #21
dylan donkersgoed commentedI pushed up a few more fixes for issues we ran into:
@RobLoach I also added a rudimentary composer.json. Please feel free to push to the MR if you need to,
Comment #22
dylan donkersgoed commentedAlso, attaching a new patch file from the latest MR.
Comment #25
ozinNew release 2.0.0 with CKEditor 5 support has been released, thanks everyone for the great work!
Comment #26
dimitrihokonmoy commentedThanks for working on this.
Works for me.Thank You
Comment #27
dimitrihokonmoy commentedComment #28
robloachThis is fantastic. Congrats on the release!
Comment #29
emptyvoid commentedAwesome work!
I'm gonna review your work as it's really unclear where to define GUI elements in the Drupal API verses the CkEditor 5 API.
I burned about a month attempting to help the management team for this module with some success.
https://www.drupal.org/project/ckeditor_bootstrap_grid
We'll see if your work can provide some light at the end of the tunnel.