Bug #26141
closedRTE removes empty paragraphs before header
0%
Description
HTMLArea Version: 2.1.6
If I save my RTE with this html Structur:
<p></p>
<p></p>
<h1>Header</h1>
One p-tag were removed:
<p></p>
<h1>Header</h1>
When I save a second time, the other one were also removed.
And so on...
removeTrailingBR isset false. But that doesn't help at all.
Updated by Mario Rimann over 12 years ago
- Target version set to 4.5.16
Just tested this one on TYPO3 v4.5.14 and it's still there. But I think it changed a bit: Instead of removing one <p> every time I save the content element, it now strips off all (empty, "empty" with just nbsp) <p> tags at the beginning of the RTE field. My test was:
<p> </p> <p> </p> <ul> <li>foo</li> <li>bar</li>
After saving:
<li>foo</li> <li>bar</li>
Updated by Bernhard Eckl over 12 years ago
Maybe it’s a configuration issue on Page TS Config?
Updated by Mario Rimann over 12 years ago
I've just tested this one on a clean TYPO3-Dummy-Installation with a brand new BE-user with no TSconfig at all -> and the above mentioned test-cases still apply. This means, that saving some stuff get's modified upon saving (e.g. empty paragraphs (with our without nbsp's in it) will get removed.
Updated by Martin Keck about 12 years ago
Same issue here with 4.5.19 and htmlArea 2.1.7
[Edit:] Strange thing is, the <p> tags only get removed (or better: are not shown) in the backend editor. In the frontend the <p> tags appear. Checked it in database: there every <p> is represented with an empty line.
Updated by Ernesto Baschny almost 12 years ago
- Category set to RTE (rtehtmlarea + ckeditor)
Please try out the fix provided in the review for #36904: https://review.typo3.org/#/c/11621/
Updated by Mario Rimann almost 12 years ago
- Target version deleted (
4.5.16)
Ernesto Baschny wrote:
Please try out the fix provided in the review for #36904: https://review.typo3.org/#/c/11621/
I tried but failed as the cherry-pick fails. So maybe the change itself is valid, but it'd need to be rebased and adapted to the current master.
I'm in for closing this issue as duplicate of #36904 as both describe the same bug IMO.
Updated by Gerrit Code Review almost 12 years ago
- Status changed from New to Under Review
Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/11621
Updated by Gerrit Code Review over 11 years ago
Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/18799
Updated by Stanislas Rolland over 11 years ago
- Status changed from Under Review to Resolved
Applied in changeset 3a2844efe275d884f96184d3c8d36ac69dab0262.