Project

General

Profile

Actions

Bug #59562

closed

WebKit may insert a span element when backspacing an heading

Added by Wouter Beeftink almost 10 years ago. Updated over 5 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 2 (0 open2 closed)

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
Actions #1

Updated by Stanislas Rolland over 9 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.

Actions #2

Updated by Stanislas Rolland over 9 years ago

  • Status changed from New to Needs Feedback
Actions #3

Updated by Stanislas Rolland over 9 years ago

In what browser do you get this behaviour?

Actions #4

Updated by Stanislas Rolland over 9 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.

Actions #5

Updated by Stanislas Rolland over 9 years ago

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

Updated by Stanislas Rolland over 9 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
Actions #7

Updated by Stanislas Rolland over 9 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
Actions #8

Updated by Stanislas Rolland over 9 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
Actions #9

Updated by Stanislas Rolland over 9 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
Actions #10

Updated by Gerrit Code Review over 9 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

Actions #11

Updated by Stanislas Rolland over 9 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
Actions #12

Updated by Gerrit Code Review over 9 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

Actions #13

Updated by Gerrit Code Review over 9 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

Actions #14

Updated by Gerrit Code Review over 9 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

Actions #15

Updated by Stanislas Rolland over 9 years ago

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

Updated by Gerrit Code Review over 9 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

Actions #17

Updated by Gerrit Code Review over 9 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

Actions #18

Updated by Stanislas Rolland over 9 years ago

  • Status changed from Under Review to Resolved
Actions #19

Updated by Wouter Beeftink over 9 years ago

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

Actions #20

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF