Project

General

Profile

Actions
Copy link

Bug #98202

open

RTE removes class from hr due to processing.yaml definition

Added by Daniel Kruse over 1 year ago. Updated over 1 year ago.

Status:
Under Review
Priority:
Should have
Assignee:
-
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
next-patchlevel
Start date:
2022-08-25
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
11
PHP Version:
7.4
Tags:
processing hr class
Complexity:
easy
Is Regression:
Sprint Focus:
Needs Decision

Description

A customer wanted to set a color to the horizontal line to change the color.
While developing i encountered the problem that the class was always removed. Debugging shows a different array for the hr-tag than for example the span, which allows at least the class.
In the typo3/sysext/rte_ckeditor/Configuration/RTE/Processing.yaml i found this special lines of code:

      hr:
        allowedAttribs:
          - class

This breaks the logic for hr-tags in typo3/sysext/core/Classes/Html/HtmlParser.php (Line 480 and following)
(btw: i'm kind of confused about the "_allowedAttribs")

a fix provided in the attached patch-file looks like this in my custom rte yaml file:

      hr:
        allowedAttribs: class

Thanks for taking a look at my first bug report

Files

local_history.patch (766 Bytes) local_history.patch Patch file Daniel Kruse, 2022-08-25 07:48
Actions
Copy link
 #1

Updated by Gerrit Code Review over 1 year ago

  • Status changed from New to Under Review

Patch set 3 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/74498

Actions
Copy link
 #2

Updated by Oliver Hader over 1 year ago

I was working on a patch months ago. The main problem is, that the RTE fails to handle these declarations correctly. In legacy code those were just strings, since TYPO3 v8 and having CKEditor4 in place, that was extended to array items as well - however, both still need to be supported.

However, currently array item declarations are not fully supported...
see https://review.typo3.org/c/Packages/TYPO3.CMS/+/74498

Actions
Copy link

Also available in: Atom PDF