- Merge request !3106Issue #3017548: "Filtered HTML" text format does not support manual teaser break (<!--break-->) โ (Open) created by smustgrave
- First commit to issue fork.
- ๐บ๐ธUnited States alison
Rebased; tests are running.
I wonder if we should update the issue title and description to "Basic HTML" instead of "Filtered HTML", based on what comes with recent versions of Drupal?
-------
Meanwhile, I tried to review the changes in DrupalPod, so far, I don't seem to be able to add<!-- break -->
to a WYSIWYG-enabled text area and have it be preserved, but I might be testing wrong -- am I meant to create a text format called filtered_html?The only config change I tried so far was to add
<!-- break -->
toeditor[settings][plugins][ckeditor5_sourceEditing]
[allowed_tags], but this change didn't "stick" (when I went back into the basic_html config form, the break tag wasn't there anymore). ANYWAY, I'm probably just not testing right -- maybe worth adding test instructions to the issue summary? - Status changed to Needs work
over 1 year ago 8:22pm 17 February 2023 - ๐บ๐ธUnited States smustgrave
Self review.
Will have to retake a look at this. Title could be updated and something may have to be tweaked as it longer appears to be working. Could use an issue summary update.
- ๐จ๐ญSwitzerland stefanos.petrakis Biel, Switzerland
Regarding #52: The provided solution is only effective for non-WYSIWYG editing. CKEditor 5 is by default removing any html comments so the reported issue will be caused by the editor and not Drupal's text formats. More on this here =>
https://ckeditor.com/docs/ckeditor5/latest/features/general-html-support...
That being said, if you would pick e.g. the RestrictedHTML format when editing which doesn't use an editor, you could test the provided solution.In general, even if a solution is presented for CKEditor (e.g. sth similar to https://ckeditor.com/docs/ckeditor5/latest/features/html-embed.html), the existing solution will still be necessary to tackle this issue.
This issue may need to change into a task, e.g. "Support manual summary delimiter breaks", where 2 problems will be addressed:
- FilterHTML's support for the delimiter (already has a solution)
- CKEditor5's support for the delimiter (as in a CKEditor Plugin that would preserve this html comment). That was also suggested in #40
- ๐ฉ๐ฐDenmark ressa Copenhagen
@alison: It's
<!--break-->
, not<!-- break -->
. I tried both, and only<!--break-->
works :)I can confirm that the "Full HTML" text format cuts off the text after
<!--break-->
, both in node display, as well as in a view. - ๐ฉ๐ฐDenmark ressa Copenhagen
I wonder if we should update the issue title and description to "Basic HTML" instead of "Filtered HTML", based on what comes with recent versions of Drupal?
Great idea, I updated to that.
For those upgrading from older versions of Drupal, it is also true that it doesn't work for "Filtered HTML" either.
Is this issue still being worked on? The last update re code is from a year ago, and I do not see any issue regarding the CKE5 issue mentioned in #56.
- ๐ญ๐บHungary Gรกbor Hojtsy Hungary
Gรกbor Hojtsy โ made their first commit to this issueโs fork.
- ๐ญ๐บHungary Gรกbor Hojtsy Hungary
I think its fine to fix the backend processing in this issue and solve the CKEditor problem in another issue. I changed the target branch of the MR to 11.x where it should be, but there are conflicts that need resolving. Hope someone can pick that up? :) In general I think the diff looks reasonable BTW.
- ๐ญ๐บHungary Gรกbor Hojtsy Hungary
BTW https://www.drupal.org/project/ckeditor_drupalbreaks โ is a contributed project that provides a UI to add the break as well as a filter to preserve the break.
- Assigned to Anita verma
- ๐จ๐ฆCanada phjou Vancouver/Paris ๐จ๐ฆ ๐ช๐บ
Just discovered that the site I was using had that patch and broke the upgrade when switching to Ckeditor5. AFter removing the patch it was working, but I guess I lose the feature.
Thanks @Gabor for the suggestion of the module, it already installed on my site, I'll try the new branch.
Do you know if we need the patch + the module or is the module enough?