Project

General

Profile

Actions

Bug #14166

closed

Content Type TABLE does not allow 0 cellspacing

Added by old_shutslar almost 20 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Content Rendering
Target version:
-
Start date:
2004-05-15
Due date:
% Done:

0%

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

Description

When setting up content type TABLE it does not allow for a cellspacing of 0. It automatically unckecks the selection and does not render this in html.

MSIE uses a default cellspacing other than 0 when rendering the tables. Therefore, there does not appear to be a way to create a table with no spacing between cells.
(issue imported from #M93)


Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Bug #24442: Each element shows start und stop 01.01.1970Closed2010-12-30

Actions
Related to TYPO3 Core - Bug #16080: Yet another table accessibility improvementClosed2006-04-24

Actions
Related to TYPO3 Core - Task #79622: Streamline CSS Styled Content and Fluid Styled ContentClosed2017-02-28

Actions
Actions #1

Updated by Ingmar Schlecht almost 20 years ago

This is about the TCA configuration of the field tt_content.table_cellspacing which is currently like this:
[type]=input
[size]=3
[max]=3
[eval]=int
[checkbox]=0
[range]
[upper]=200
[lower]=0
[default]=0

This would obviously have to be changed in some way, I think.

Anyone with more knowledge on what could be changed without causing any sideeffects please comment!

Actions #2

Updated by Robert Lemke almost 20 years ago

What value would you like to put into that field?

I admit that it is quite confusing that the zero (0) disappears as soon as you leave the field. But this is the correct behaviour, because 0 is the default value, better: the "checkbox value" (see: http://typo3.org/documentation/document-library/doc_core_api/columns_fieldnam-1/)

This means that if the cellspacing or cellpadding field is empty, in reality "0" is set, because the table_cellspacing and table_cellpadding fields in MySQL are of the type integer with default to zero.

As long as you don't want to put something else than an integer value here, I wouldn't consider it a bug. But maybe it would be nicer to display the "0" when the checkbox is checked.

Actions #3

Updated by Ernesto Baschny over 18 years ago

Both fields should allow an int value OR an empty value ("", i.e. NULL). So TYPO3 could be differenciated if the user doesn't want to set the field (NULL), which should render a TABLE without any "cellspacing" (or "cellpadding") attribute or if he wants to set the field with a value of "0" (which would render cellpadding/spacing="0").

As far as I can see there is no TCA-way of having an input field that has "eval=int" but still allows a NULL value. So I guess this would require the new possibility in the CORE first. But maybe I just cannot see how it could be acomplished (http://typo3.org/documentation/document-library/doc_core_api/columns_fieldnam-1/).

Actions #4

Updated by Benni Mack about 14 years ago

Hey Ernesto,

we could change the DB field to be varchar and then we do the int-evaluation on TCA basis, having finally two values to differentiate ("" and 0) in the DB.

Of course, the cleanest way would be that TYPO3 finally learns to understand the difference between 0 and NULL.

Actions #5

Updated by Chris topher almost 14 years ago

Related to #16080.

Actions #6

Updated by Riccardo De Contardi almost 11 years ago

  • Target version deleted (0)

Tthe bug is still present in TYPO3 CMS 6.1.0
Said that, AFAIK table attributes like cellspacing and cellpadding are deprecated in HTML5
Honeslty, I think they should be removed.

The field "Additional CSS Class" trims white spaces, so you can't add more than one CSS class... please, fix it!

THANKS

Actions #7

Updated by Felix Kopp over 9 years ago

  • Description updated (diff)
  • Parent task set to #61842
  • TYPO3 Version changed from 3.6.0 to 6.2
  • PHP Version deleted (4)
  • Is Regression set to No
Actions #8

Updated by Riccardo De Contardi over 9 years ago

The bug is still present in TYPO3 6.2.5;

The "Additional CSS Class" field now does not trim spaces

Actions #9

Updated by Riccardo De Contardi almost 7 years ago

- on 7.6.16 has no more the fields "border", "cellpadding", "cellspacing" if you use Fluid Styled Content. They are still present if you use CSS Styled Content.

- on latest master (8.7.1) the TABLE CE has no more the fields "border", "cellpadding", "cellspacing" as far as I can see (both CSS Styled Content and Fluid Styled Content - I have not tested with Compatibility7 loaded, though).

- The issue about the distinction between 0 and NULL is still present AFAIK.

Actions #10

Updated by Riccardo De Contardi almost 7 years ago

  • Parent task deleted (#61842)
Actions #11

Updated by Georg Ringer over 6 years ago

  • Related to Task #79622: Streamline CSS Styled Content and Fluid Styled Content added
Actions #12

Updated by Georg Ringer over 6 years ago

  • Status changed from New to Closed

with #79622 the fields have been removed. no action will be done for 7.6 anymore. therefore I am closing this issue.

Actions

Also available in: Atom PDF