Bug 41063 - [test case] colspan=0 and rowspan=0 is not treated correctly
Summary: [test case] colspan=0 and rowspan=0 is not treated correctly
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: khtml renderer (show other bugs)
Version: 3.0
Platform: RedHat Enterprise Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
: 120215 (view as bug list)
Depends on:
Blocks:
 
Reported: 2002-04-15 20:18 UTC by dath
Modified: 2009-05-05 06:06 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
colspan="0" doesn't work in strict mode in konqueror (288 bytes, text/html)
2004-11-19 14:56 UTC, Yan Morin
Details
rowspan="0" and colspan="0" test case (818 bytes, text/html)
2004-11-19 15:00 UTC, Yan Morin
Details

Note You need to log in before you can comment on or make changes to this bug.
Description dath 2002-04-15 20:14:54 UTC
(*** This bug was imported into bugs.kde.org ***)

Package:           khtml
Version:           kdeaddons-konqueror-3.0.0-1 (using KDE 3.0.0 )
Severity:          normal
Installed from:    RedHat RPMs
Compiler:          Not Specified
OS:                Linux
OS/Compiler notes: Not Specified

According to the w3c "colspan=0" in a "<td>" tag should force the given cell to span all remaining cells in the table.  In the example I gave below you should see a row with three cells followed by a row with a cell spanning all three cells.  Konquerer does not display this properly.

The following example will reproduce this bug.

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head><title>colspan test</title></head>
<body>

<p>
<table border>
<tr><td>cell 1</td><td>cell 2</td><td>cell 3</td></tr>
<tr><td colspan=0 align=center>Center</td></tr>
</table>
</p>

</body>
</html>

(Submitted via bugs.kde.org)
Comment 1 Stephan Kulow 2003-10-31 16:57:41 UTC
renders exactly like IE and mozilla
Comment 2 Yan Morin 2004-11-19 14:56:34 UTC
Created attachment 8333 [details]
colspan="0" doesn't work in strict mode in konqueror

Same file, but with a strict Doctype
Comment 3 Yan Morin 2004-11-19 15:00:44 UTC
Created attachment 8334 [details]
rowspan="0" and colspan="0" test case

Should look like:
 ________
|________|
|  |__|__|
|  |__|__|
|__|__|__|
Comment 4 Eric Kjeldergaard 2006-02-01 18:54:07 UTC
Could this please be reopened?  I am on ubuntu's 3.5.1 release and am still noticing this incorrect non-standard  ( http://www.w3.org/TR/html4/struct/tables.html#h-11.2.6 ) behaviour.  The bottom test case here is an excellent example of the problem.  This bug is the same as http://bugs.kde.org/show_bug.cgi?id=120215 , it appears.  It would be nice if our wonderful khtml would render this properly.
Comment 5 Eric Kjeldergaard 2006-02-01 18:57:51 UTC
oh, I guess I can reopen this.  I hope I'm not stepping on any toes by doing it.  I'm marking 120215 a dup of this...
Comment 6 Eric Kjeldergaard 2006-02-01 18:57:57 UTC
*** Bug 120215 has been marked as a duplicate of this bug. ***
Comment 7 dath 2006-02-01 19:14:17 UTC
I agree, this is still a problem.  I tried the above attachment (id=8334) in Firefox 1.0.7 and it worked just fine, but in Konqueror 3.5.0 on FC4 it still is broken, not taking up all columns in the top row.
Comment 8 Allan Sandfeld 2007-09-25 15:00:35 UTC
I am letting the bug stay, but for me this is a WONTFIX issue. 

I've fixed another bug where we didn't handle colspan=0 exactly like Firefox and MSIE. Now we do, but it is nothing like the W3C standard, only the de-facto standard.
Comment 9 Eric Stith 2007-09-26 00:41:01 UTC
If you open the examples posted in comments #2 and #3 in Konqueror and essentially anything else (firefox, ie, opera), you will see that the only one that is different is Konqueror.  This de-facto standard does not exist, and the W3C agrees with them.
Comment 10 Allan Sandfeld 2007-09-27 22:26:32 UTC
Take a look at Opera rendering again, only rowspan=0 is handled.

Firefox seems to have changed handling of colspan=0 recently unless my memory is playing tricks on me.
Comment 11 Jiri Palecek 2007-09-28 00:44:08 UTC
On Thu, 27 Sep 2007 22:26:37 +0200, Allan Sandfeld <kde@carewolf.com>  
wrote:

> Take a look at Opera rendering again, only rowspan=0 is handled.
>
> Firefox seems to have changed handling of colspan=0 recently unless my  
> memory is playing tricks on me.


Had or had it not played tricks, the the html standard says this:

colspan = number [CN]
This attribute specifies the number of columns spanned by the current  
cell. The default value of this attribute is one ("1"). The value zero  
("0") means that the cell spans all columns from the current column to the  
last column of the column group (COLGROUP) in which the cell is defined.

What do you want more?
Comment 12 Allan Sandfeld 2007-09-28 07:48:22 UTC
Besides being in the standard it has to be safe to implement as well. This seems safe since Firefox can live with it, so we only loose a little compability with MSIE who we match right now.
Comment 13 George Goldberg 2008-04-06 03:19:43 UTC
bug still present in svn trunk r793457 using testcase from comment #3
Comment 14 Carlos Licea 2009-05-05 06:06:31 UTC
 SVN commit 963656 by clicea:
 
 Finally respect the span = 0 properly as described by the HTML4.01 standard. 
 Closing a bug from 2002!.
 BUG:41063
 
 
  M  +4 -4      html/html_tableimpl.cpp  
  M  +155 -34   rendering/render_table.cpp  
  M  +8 -0      rendering/render_table.h  
 
 
 WebSVN link: http://websvn.kde.org/?view=rev&revision=963656