Make allowed tags configurable

Created on 29 June 2022, almost 3 years ago

Updated 30 August 2023, over 1 year ago

Problem/Motivation

Currently the allowed tags are hard-coded:

  /**
   * List of allowed tags.
   *
   * @var array
   */
  protected $allowedTags = [
    '<drupal-render-placeholder>',
    '<embed>',
    '<hr>',
    '<iframe>',
    '<img>',
    '<input>',
    '<link>',
    '<object>',
    '<script>',
    '<source>',
    '<style>',
    '<video>',
  ];

Let's see if there's real demand to make them configurable. I'll postpone this for real-world feedback from users.

Steps to reproduce

Proposed resolution

Save this to a config variable. As https://www.php.net/manual/de/function.strip-tags.php allows a string as second parameter, we could use a string instead of the array for simplification then.

Remaining tasks

User interface changes

API changes

Data model changes

✨ Feature request

Status

Postponed

Version

1.0

Component

Code

Created by

🇩🇪Germany Anybody Porta Westfalica

Live updates comments and jobs are added and updated live.

Incomplete comments

Comments & Activities

Not all content is available!

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

Comment about 2 years ago →
🇺🇦Ukraine ruslan piskarov Kyiv, Ukraine
Ruslan Piskarov → made their first commit to this issue’s fork.
@ruslan-piskarov opened merge request.
Comment over 1 year ago →
🇬🇧United Kingdom AaronMcHale Edinburgh, Scotland
We have a use-case where we need to use this on a sidebar region, but exclude the <h2> tag because we have the title of the menu block always hidden, but still rendered as visually-hidden. So excluding that from determining if the region should be shown would be necessary.
Comment over 1 year ago →
🇺🇦Ukraine ruslan piskarov Kyiv, Ukraine
What about case if you use any block with text field and CKEditor and applied H2 tag by CKEditor?

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024