Bug #16080
Yet another table accessibility improvement
| Status: | Closed | Start date: | 2006-04-24 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | - | % Done: | 0% | |
| Category: | Content Rendering | Spent time: | - | |
| Target version: | - | |||
| TYPO3 Version: | 4.5 | Is Regression: | ||
| PHP Version: | Sprint Focus: | |||
| Complexity: | 
Description
Hi,
This is connected to the #2405 report. I've posted it here to be in the correct category.
It's the patch requested by Sebastian Kurfuerst.
The purpose is to correct 2 features :
- be able to specify a 0 value in the border/cellapdding/cellspacing when creating a table
- add the row numbering, even for the last
--- class.tx_cssstyledcontent_pi1.orig.php    2006-04-24 09:18:49.000000000 0200
++ class.tx_cssstyledcontent_pi1.php    2006-04-24 09:19:27.000000000 0200@ -213,7 +213,7 @
                 }
                 if (!$noStyles)    {
                     $oddEven = $k%2 ? 'tr-odd' : 'tr-even';
-                    $rowAttribs =  ($k>0 && ($rCount-1)==$k) ? ' class="'.$oddEven.' tr-last"' : ' class="'.$oddEven.' tr-'.$k.'"';
                    $rowAttribs =  ($k>0 && ($rCount-1)==$k) ? ' class="'.$oddEven.' tr-last tr-'.$k.'"' : ' class="'.$oddEven.' tr-'.$k.'"';
                 }
                 $rows[$k]='
                     <tr'.$rowAttribs.'>'.implode('',$newCells).'@ -849,9 +849,9 @
// Create table attributes array:
         $tableTagParams = array();
-        $tableTagParams['border'] =  $this->cObj->data['table_border'] ? intval($this->cObj->data['table_border']) : $tableTagParams_conf['border'];
-        $tableTagParams['cellspacing'] =  $this->cObj->data['table_cellspacing'] ? intval($this->cObj->data['table_cellspacing']) : $tableTagParams_conf['cellspacing'];
-        $tableTagParams['cellpadding'] =  $this->cObj->data['table_cellpadding'] ? intval($this->cObj->data['table_cellpadding']) : $tableTagParams_conf['cellpadding'];
+        $tableTagParams['border'] =  $this->cObj->data['table_border'] >= 0 ? intval($this->cObj->data['table_border']) : $tableTagParams_conf['border'];
+        $tableTagParams['cellspacing'] =  $this->cObj->data['table_cellspacing'] >= 0 ? intval($this->cObj->data['table_cellspacing']) : $tableTagParams_conf['cellspacing'];
+        $tableTagParams['cellpadding'] =  $this->cObj->data['table_cellpadding'] >= 0 ? intval($this->cObj->data['table_cellpadding']) : $tableTagParams_conf['cellpadding'];
         $tableTagParams['bgcolor'] =  isset($conf['color.'][$this->cObj->data['table_bgColor']]) ? $conf['color.'][$this->cObj->data['table_bgColor']] : $conf['color.']['default'];// Return result:
(issue imported from #M3325)Related issues
History
#1 Updated by Chris topher over 5 years ago
Related to #93.
#2 Updated by Ernesto Baschny about 3 years ago
- Status changed from Accepted to Needs Feedback
- Assigned To deleted (Ernesto Baschny)
- Target version deleted (0)
- TYPO3 Version set to 4.5
Is this still an issue? Sorry for taking so long, but I unassign myself because I don't really have this problem.
#3 Updated by Alexander Opitz over 2 years ago
- Status changed from Needs Feedback to Closed
No response over 9 month => closed
