- Issue created by @longwave
- Status changed to Needs review
7 months ago 5:38pm 25 April 2024 - Status changed to Needs work
7 months ago 6:07pm 25 April 2024 The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- Status changed to Needs review
7 months ago 7:00pm 25 April 2024 - Status changed to Needs work
7 months ago 7:01pm 25 April 2024 - Status changed to Needs review
7 months ago 7:24am 26 April 2024 - 🇬🇧United Kingdom longwave UK
Fixed #7, also backported the .gitignore from 11.x to prevent this happening again, this isn't the first time I've done this by mistake when switching branches.
- Status changed to RTBC
7 months ago 9:20pm 26 April 2024 - 🇺🇸United States smustgrave
Tested this out on a standard install on 11.x
Enabled Media library to test that button.
Updated text format with media library button, image upload, table, lists
Created a page testing all the buttons behave as expected. - Status changed to Needs work
7 months ago 10:04am 27 April 2024 - 🇧🇪Belgium wim leers Ghent 🇧🇪🇪🇺
Yay, tests are passing — no subtle BC breaks this time 🥳
Per https://github.com/ckeditor/ckeditor5/releases/tag/v41.3.0, 41.3 brings multi-level lists and improved keyboard navigation.
Review
Nit:
package.json
lists41.3.0
, not41.3.1
. - First commit to issue fork.
- 🇮🇳India adwivedi008
Rebased '3443567-update-ckeditor-5-11.x' branch with 11.x
- Status changed to RTBC
7 months ago 10:11am 29 April 2024 - 🇵🇱Poland salmonek
41.3 brings multi-level lists (in a separate plugin which we're not offering,
We'll have it covered in the CKEditor 5 Premium Features module.
- Status changed to Needs work
7 months ago 1:38pm 29 April 2024 - Status changed to RTBC
7 months ago 1:51pm 29 April 2024 - 🇬🇧United Kingdom longwave UK
It's using the tilde syntax so IMHO it doesn't really matter, but updated anyway.
- 🇬🇧United Kingdom catch
Oh good point, should've spotted that but thanks for updating anyway.
Committed/pushed to 11.x and cherry-picked to 10.3.x, thanks!
- Status changed to Fixed
7 months ago 2:57pm 29 April 2024 Automatically closed - issue fixed for 2 weeks with no activity.