-
-
Notifications
You must be signed in to change notification settings - Fork 227
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 5 #6249
base: master
Are you sure you want to change the base?
CKEditor 5 #6249
Conversation
NOTE: Requires changes found in #6248 in order to work correctly. |
8edaf2e
to
910625d
Compare
Do you want to merge this into master for eventual release in 2.1? That way people could start testing it more easily? |
We're working on it. |
b631e67
to
264d140
Compare
Lots of updates today. |
based on this post here, as well as my experience, the vertical height of the window can no longer be controlled via the i have added the following line to one of the global stylesheets which seems to work fine. .ck-editor__editable {
min-height: 250px;
} |
Thanks for digging that out. I was wondering what direction to take there. Perhaps we should also set |
i like the idea of having both ( as to which css file you want to include them in... well i have no opinion on that one... |
Tested on versions v210, v200, v158, v157, v156 to confirm that the CKE5 editor loads correctly using the code changes in this PR.
(Note on v156 the stylesheet is in the includes dir, not in the css subdir) |
In Chrome 126.0.6478.116, CKEditor fails to initialize (remains as default textarea) and see this in the console:
|
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
Thanks for your patience. It appears that it doesn't like having a I've pushed an update which seems to work fine now with single-language sites. Tested on v158 and v210.
|
This comment was marked as resolved.
This comment was marked as resolved.
01dd777
to
e2c9aed
Compare
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
Many thanks all, working fine now. |
Anyone tried 1.5.7d? |
It is working fine on my end now. Thanks for everyone's help. |
|
Anyone tried 1.5.7d?
Yes.
|
I have two questions regarding ck5, not sure if it is a good place to post here... In the previous version ck4, it seems it has a special right-click menu. So under the editing page, if I right click my mouser, it will show a special menu, to replace the menu from the browser. For example, if I right click the any text, it will show Cut, Copy and Paste. If I right click anywhere in a table, it will show Cell, Row, Column, Delete table and Table properties. In new version ck5, it seems this function is gone. If I right click anywhere under the editing page, it will only show the default options from the browser. Also, it seems I cannot find a place to edit table prosperities now... For ck4, if I double click any hyperlinks, it will pop up a window with two tabs, one for link info, another for target. Under target tab, I can set the action to open a new window (_blank), or same window (_self). I cannot find this function in ck5 as well... In ck5, if I double click any hyperlinks, I'm only able to either change the link, or remove the link |
I just tried this demo on ck websites (https://ckeditor.com/ckeditor-5/demo/feature-rich/).
|
I tested and it seems that by default Ckeditor5 adds target="_blank" and rel="noopener noreferrer" to links, so you would have to have the source button available to get in and edit. |
11f7c3a
to
218e464
Compare
Correct: This is no longer a feature in CKEditor v5. Fortunately the browser's context-menu already allows copy/paste by default. |
I've updated it to include the Table Properties editor, and an option to specify |
@torvista If you have some time, do you mind testing multi-language stuff? |
Steve is on holiday till @ 11th or 12th |
Ref #6209
This uses CKEditor 5 "Classic Mode", which turns a textarea field into an in-browser editor field.
Minor differences from CKE4:
Tested on versions v210, v200, v158, v157, v156 to confirm that the CKE5 editor loads correctly using the code changes in this PR.
To backport to an older version, simply copy the files from this PR.
/admin/includes/ckeditor.php
(whole file)/editors/ckeditor5/config.js
(whole file and directory)and manually patch the stylesheet:
/admin/includes/css/stylesheet.css
(just the last few lines related to .ck-editor__editable must be copied into the old file) (Note on v156 the stylesheet is in the/admin/includes
dir, not in the/admin/includes/css
subdir)