Contrib.social

'Limit allowed HTML tags and correct faulty HTML' remove custom allowed HTML Tags

Open on Drupal.org →
Created on 14 December 2016, over 8 years ago
Updated 5 April 2024, about 1 year ago

I created a custom ckeditor plugin (button) that was included in the basic html Text format where the filter "Limit allowed HTML tags and correct faulty HTML" is enable. My plugin Include a custom HTML tag <custom-embed-jwplayer> and this custom tag was added in the "allowed tag" text field, but the tag is removed in the processed body.

"body": [
    {
      "value": "<custom-embed-jwplayer class=\"jwplayer\" data-video-id=\"THE_VIDEO_ID\"><img src=\"https://assets-jpcust.jwpsrv.com/thumbs/THE_VIDEO_ID.jpg\" /></custom-embed-jwplayer>",
      "format": "basic_html",
      "summary": "",
      "processed": "<img src=\"https://assets-jpcust.jwpsrv.com/thumbs/THE_VIDEO_ID.jpg\" />"
    }
  ],

It should not happend because the tag is included in the field of allowed tags, nevertheless it could happend because it is a custom html tag and could be taken like a fautly HTML

🐛 Bug report
Status

Active

Version

11.0 🔥

Component
CKEditor 5  →

Last updated about 20 hours ago

Created by

🇨🇴Colombia apupiales

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • Comment about 1 year ago →
    🇦🇺Australia pameeela
  • Status changed to Postponed: needs info about 1 year ago
  • Comment about 1 year ago →
    🇦🇺Australia pameeela

    If I use this filter on a non-CKE field, the custom tags output as expected, which suggests this is not a Drupal issue.

    So I am guessing that CKE's default filtering only recognises regular HTML. I would guess there is a way to override it to in CKE config?

  • Comment 4 months ago →
    🇦🇺Australia griffynh Sydney

    Hola, this came up in the #bugsmash channel as the PMNMI daily triage target.

    Since we haven't had an update in eight months, if there is no update in the next three months, this issue may be closed.

  • Status changed to Closed: outdated 10 days ago
  • Comment 10 days ago →
    🇺🇸United States smustgrave

    Since there hasn't been a follow up going to close out for now. If still an issue in D11 please re-open.

    Thanks all

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024