-
-
Notifications
You must be signed in to change notification settings - Fork 135
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ckeditor upgrade #3879
base: 8.3
Are you sure you want to change the base?
Ckeditor upgrade #3879
Conversation
Configuration/Settings.yaml
Outdated
@@ -94,7 +94,7 @@ Neos: | |||
################################# | |||
# INTERNAL CONFIG (no API) | |||
################################# | |||
frontendDevelopmentMode: false | |||
frontendDevelopmentMode: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here you accidentally committed a config. BTW, you can create a config file in your project and change the setting there instead of changing it in the package itself ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks :). I removed the change from the PR.
packages/neos-ui/package.json
Outdated
"@fortawesome/free-regular-svg-icons": "^5.15.3", | ||
"@fortawesome/free-solid-svg-icons": "^5.15.3", | ||
"@ckeditor/ckeditor5-highlight": "42.0.2", | ||
"@fortawesome/fontawesome-svg-core": "^6.5.2", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it intended that you upgrade CKEditor and font awesome at once?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed the fontawesome update :).
What I did
I increased the CKEditor5 version to 42.0.2 from version 16.0.0.
Also I fixed small build-errors, which occured by upgrading the CKEditor packages.
** How I did it**
How to verify it
Checkout the PR and use the Neos backend.
Already tested Plugins: