Bug #59562

WebKit may insert a span element when backspacing an heading

Added by Wouter Beeftink over 7 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Should have
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
-
Start date:
2014-06-13
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

When removing a block format (h2, p) from an element in the RTE, the following element (for example the next paragraph) gets an inline span and inline CSS. That shouldn't happen.

To reproduce you can do the following:

  1. Create a content element (for example "text").
  2. Add a header with some text.
  3. Add a paragraph with some text after the header.
  4. Clear the header using the backspace key.
  5. Delete the empty line left by the header using the delete key.
  6. The paragraph is now bold and contains a spam with inline CSS.

Because this is quite specific I also attached an animated gif that hopefully shows this issue in a way that's easier to understand.

I can reproduce this issue in TYPO3 6.1.7 and 6.2.2. I also seem to be able to reproduce this issue using any sort of block format.


Files

rte-issue.gif (341 KB) rte-issue.gif Wouter Beeftink, 2014-06-13 17:30

Related issues

Related to TYPO3 Core - Bug #52470: In Chrome RTE inserts weird SPAN tagsClosedStanislas Rolland2013-10-02

Actions
Related to TYPO3 Core - Bug #67348: RTE inserts <span class="line-height: xx"> in ChromeClosed2015-06-08

Actions
#1

Updated by Stanislas Rolland about 7 years ago

In what browser do you get this behaviour?

I am unable to reproduce this with Firefox 32.0.3.

Also unable to reproduce with Google Chrome 38.0.2125.101.

#2

Updated by Stanislas Rolland about 7 years ago

  • Status changed from New to Needs Feedback
#3

Updated by Stanislas Rolland almost 7 years ago

In what browser do you get this behaviour?

#4

Updated by Stanislas Rolland almost 7 years ago

I was able to reproduce this behaviour with Google Chrome 39.0.2171.99.

However, the header is not deleted. The contents of the paragraph rather become integrated into the header.

#5

Updated by Stanislas Rolland almost 7 years ago

  • Status changed from Needs Feedback to Accepted
  • Assignee set to Stanislas Rolland
#6

Updated by Stanislas Rolland almost 7 years ago

  • Subject changed from Removing a block format inserts spans and inline CSS to Removing a block format inserts spans and inline CSS in Chrome
#7

Updated by Stanislas Rolland almost 7 years ago

  • Subject changed from Removing a block format inserts spans and inline CSS in Chrome to Backspacing an empty block inserts spans and inline CSS in Chrome
#8

Updated by Stanislas Rolland almost 7 years ago

  • Subject changed from Backspacing an empty block inserts spans and inline CSS in Chrome to Backspacing an empty block inserts a span tag in Chrome
#9

Updated by Stanislas Rolland almost 7 years ago

  • Subject changed from Backspacing an empty block inserts a span tag in Chrome to WebKit may insert a span element when backspacing an empty block
#10

Updated by Gerrit Code Review almost 7 years ago

  • Status changed from Accepted 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 http://review.typo3.org/36035

#11

Updated by Stanislas Rolland over 6 years ago

  • Subject changed from WebKit may insert a span element when backspacing an empty block to WebKit may insert a span element when backspacing an heading
#12

Updated by Gerrit Code Review over 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 http://review.typo3.org/36035

#13

Updated by Gerrit Code Review over 6 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36035

#14

Updated by Gerrit Code Review over 6 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36035

#15

Updated by Stanislas Rolland over 6 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
#16

Updated by Gerrit Code Review over 6 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36086

#17

Updated by Gerrit Code Review over 6 years ago

Patch set 2 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36086

#18

Updated by Stanislas Rolland over 6 years ago

  • Status changed from Under Review to Resolved
#19

Updated by Wouter Beeftink over 6 years ago

Thanks Stanislas! I'm looking forward to seeing the change in the next TYPO3 version.

#20

Updated by Benni Mack about 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF