Project

General

Profile

Actions

Bug #101869

closed

Epic #99669: CKEditor5 Collection

Story #101905: CKEditor4 to CKEditor5 Migration Issues

CKEditor5: Pass full yaml editor configuration to CKEditor5

Added by Georg Tiefenbrunn 10 months ago. Updated 9 months ago.

Status:
Resolved
Priority:
Should have
Assignee:
-
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
-
Start date:
2023-09-07
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
12
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

I just tried setting editor.config.toolbar.shouldNotGroupWhenFull = false (RTE yaml configuration) but this value is hardcoded in CKEditor5Migrator::migrateToolbar()

Instead of using the CKeditor 5 setting editor.config.toolbar.removeItems (https://ckeditor.com/docs/ckeditor5/latest/features/toolbar/toolbar.html#extended-toolbar-configuration-format) you have to use CKeditor 4 setting editor.config.removeButtons.

What about adding an option to disable the CKEditor4 to CKEditor5 migration and simply add the full yaml configuration (except those settings which cant't be defined via yaml like content and UI language) to CKEditor5?


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #100784: CKEditor 5 - Configuration of custom CKeditor 5 plugin via RTE.editor.config.{customPlugin} not possibleResolved2023-05-10

Actions
Actions #1

Updated by Georg Tiefenbrunn 10 months ago

  • Related to Bug #100784: CKEditor 5 - Configuration of custom CKeditor 5 plugin via RTE.editor.config.{customPlugin} not possible added
Actions #2

Updated by Georg Tiefenbrunn 10 months ago

  • Tracker changed from Feature to Bug
  • TYPO3 Version set to 12

https://docs.typo3.org/c/typo3/cms-rte-ckeditor/12.4/en-us/Configuration/Reference.html#editor

[...] All CKEditor-specific options, which one could imagine are available under “config” property and handed over to CKEditor’s instance-specific config array.

As documented in #100784 only some configuration keys are forwarded.
editor.config.toolbar.* in only partly forwarded.

Actions #3

Updated by Georg Tiefenbrunn 10 months ago

  • Description updated (diff)
Actions #4

Updated by Benjamin Franzke 9 months ago

  • Parent task set to #99669
Actions #5

Updated by Benjamin Franzke 9 months ago

  • Parent task changed from #99669 to #101905
Actions #6

Updated by Gerrit Code Review 9 months ago

  • Status changed from New to Under Review

Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/81025

Actions #7

Updated by Gerrit Code Review 9 months ago

Patch set 3 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/78959

Actions #8

Updated by Gerrit Code Review 9 months ago

Patch set 1 for branch 12.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/81000

Actions #9

Updated by Stefan Froemken 9 months ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions

Also available in: Atom PDF