Bug #86862
closedUsing <f:spaceless> in layout for rendering any fluid_styled_content element causes valid RTE content to be be destroyed.
100%
Description
As pointed out here: #80695 <f:spaceless> removes too much whitespace in certain contexts.
Important here is, that RTE content potentially is such context.
Therefore I suggest to remove the <f:spaceless> from the layout
Updated by Helmut Hummel about 6 years ago
- Related to Bug #80695: f:spaceless ViewHelper removes too many spaces added
Updated by Helmut Hummel about 6 years ago
Example:
<strong>my bold text</strong> <em>with a space to the emphasize</em>
note, that it is totally valid in RTE context to have two different inline formattings separated with a space
Updated by Markus Klein about 6 years ago
I agree with that!
There is no other sane solution to solve this.
Updated by Gerrit Code Review about 6 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58858
Updated by Markus Klein about 6 years ago
- Related to Bug #85777: <f:spaceless> eats too many blank spaces added
Updated by Markus Klein about 6 years ago
- Related to deleted (Bug #85777: <f:spaceless> eats too many blank spaces)
Updated by Markus Klein about 6 years ago
- Has duplicate Bug #85777: <f:spaceless> eats too many blank spaces added
Updated by Markus Klein about 6 years ago
- Related to deleted (Bug #80695: f:spaceless ViewHelper removes too many spaces)
Updated by Markus Klein about 6 years ago
- Has duplicate Bug #80695: f:spaceless ViewHelper removes too many spaces added
Updated by Benjamin Kott about 6 years ago
Removing the spaceless viewhelper is a massive change in behaviour during released versions that WILL lead to visual differences. Please see very very simple example: https://codepen.io/benjaminkott/pen/dQYwwy
This will make the change breaking, and so the change must atleast be marked as beeing that. If you rely on these to reduce for example spacing between buttons or other examples. You will need to adapt your templates and override the layout if you are currently not doing this.
In addition we reintroduce a lot of unnessesary bloat we tried to remove in the first place. That means if you are fine with the current solution you will now need to take additional actions to optimize the html output.
Means, whatever we will change, it will be breaking in some kind or another. This is why i prefer not to make any backports to older versions in this area.
But since v9.5 is almost fresh out of the door we can/should make an exception for this one. But i would still like to see an option that keeps the compression and the removal of bloat, but is handling block and inline elements differently/better.
Updated by Helmut Hummel about 6 years ago
Benjamin Kott wrote:
Please see very very simple example: https://codepen.io/benjaminkott/pen/dQYwwy
Well, yes. inline-block elements must not have spaces to avoid gaps when gaps are not wanted. In RTE context (or any context where basically editors control the HTML), inline-block elements must have spaces
I tend to say that situations where spaces need to be removed between inline-block elements are not controlled by editors, right?
I at least can't imagine a case where editors add a space and this space should be trimmed.
So when you don't want a CSS fix for this issue, you have to adapt your templates to not have gaps where you don't want one. But CSS fixes might be a tradeoff:
https://stackoverflow.com/questions/5078239/how-do-i-remove-the-space-between-inline-block-elements
But i would still like to see an option that keeps the compression and the removal of bloat, but is handling block and inline elements differently/better.
If I'm not completely mistaken, it is impossible to decide on HTML level where to trim and where not (HTML parsing is hard enough), but since CSS defines which tag should be displayed as inline block,
we can't do anything on HTML level.
Updated by Markus Klein about 6 years ago
Lets fix the 90% case. I would argue the space stuff within RTE is affecting more people than the button stuff. But maybe I'm wrong about that. Thinking of our customers I see mostly see the argument "I want to control the output of my text myself".
Updated by Gerrit Code Review almost 6 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58858
Updated by Riccardo De Contardi over 5 years ago
- Related to Bug #88203: RTE CKEditor removes spaces between formatted words added
Updated by Andreas Kießling over 5 years ago
Any news on that issue? We just had to override the layout to get the whitespace back..
Updated by Gerrit Code Review over 5 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/58858
Updated by Georg Ringer over 5 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset b72e2a51d5a9a50e3752c8bd8ba546170ec2a8c8.
Updated by Benni Mack almost 5 years ago
- Status changed from Resolved to Closed
Updated by Riccardo De Contardi over 4 years ago
- Related to Bug #91003: Spaces between adjacent hyperlinks in Regular Text Element content elements do not appear when the content is rendered added