Project

General

Profile

Actions

Bug #16171

closed

Inconsistent markup when adding tr/td in different ways (Firefox)

Added by Uschi Renziehausen over 18 years ago. Updated over 16 years ago.

Status:
Closed
Priority:
Should have
Category:
RTE (rtehtmlarea + ckeditor)
Target version:
-
Start date:
2006-05-22
Due date:
% Done:

0%

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

Description

Hi Stanislas,

while playing around with those marvelous table features I discovered a little beast when using Firefox. System information first: Typo3 4.0 final, rtehtmlarea 1.4.1, Firefox 1.07 running under windows 2000, server under gentoo.

In FF, there are two ways to insert a new row or cell, either via those cute little arrows or using the context menu and then choosing insert row/cell before/after.

Unfortunately the resulting markup is inconsistent:

When adding a new row via context menu -> insert row/cell before/after the editor produces the following code (html-view):
<tr><td>content of cell</td></tr>

But when I add a new row via those neat little arrows, I a style="vertical-align: top" for every cell:
<tr><td style="vertical-align: top">content of cell</td></tr>

Could you perhaps kill the unwanted style attribute.

Thank you very much for all the work you have been doing for us all,

Uschi

(issue imported from #M3537)


Files

rtehtmlarea_bugfix_3537.patch (2.49 KB) rtehtmlarea_bugfix_3537.patch Administrator Admin, 2008-02-19 17:23
rtehtmlarea_bugfix_3537_2.patch (3.12 KB) rtehtmlarea_bugfix_3537_2.patch Administrator Admin, 2008-02-20 04:27
Actions #1

Updated by Stanislas Rolland almost 17 years ago

I am thinking of disabling these handles altogether. What do you think?

Actions #2

Updated by Stanislas Rolland almost 17 years ago

execCommand( 'enableInlineTableEditing', false, false)

Actions #3

Updated by Stanislas Rolland almost 17 years ago

The Firefox handles will be disabled by default. The may be enabled in Page TSConfig with property:

RTE.default.buttons.table.enableHandles = 1

Fixed in SVN TYPO3core branch TYPO3_4-2 revision 3233.

Actions #4

Updated by Stanislas Rolland almost 17 years ago

A second patch is required because execCommand requires a boolean argument.

Actions #5

Updated by Stanislas Rolland almost 17 years ago

Second patch committed to SVN TYPO3core branch TYPO3_4-2 revision 3235.

Actions

Also available in: Atom PDF