[BC] border-collapse problems at cellmap holes

RESOLVED FIXED in Future

Status

()

Core
Layout: Tables
P3
normal
RESOLVED FIXED
16 years ago
4 years ago

People

(Reporter: Bernd, Unassigned)

Tracking

(Depends on: 1 bug, {testcase})

Trunk
Future
x86
Windows 98
testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 3 obsolete attachments)

(Reporter)

Description

16 years ago
see attached testcase there should be a horizontal line below the third cell in
the first row.
(Reporter)

Comment 1

16 years ago
Created attachment 112619 [details]
testcase

Comment 2

16 years ago
If you drag a window over the border-collapse:collapse'd table, you'll notice
that as you move it over and then away from the table, then the border lines
seem to display all kinds of weird repaintings (of varying length).

I had similar problems with borders being in the wrong place/not in the correct
place that I reported in bug 157047.
Keywords: testcase

Updated

15 years ago
Priority: -- → P3
Target Milestone: --- → Future

Comment 3

15 years ago
just rediscovered this bug.  I had put together another testcase before finding
this bug.   I'll attach it just for laughs I guess.

Comment 4

15 years ago
Created attachment 133798 [details]
testcase

Updated

14 years ago
Depends on: 203686

Comment 5

13 years ago
Created attachment 189136 [details]
testcase

The bug's still around.  I just spent an hour or two trying to narrow it down &
decided to add my test case.
I cannot reproduce this on 2006012105-trunk/WinXP.

-> WFM (or dup of bug 147135?)
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → WORKSFORME
(Reporter)

Comment 7

12 years ago
The testcase that I wrote is still wrong, Masayuki before you mark a bug as WFM please make sure that you understand what the bug is about. This bug describes a problem that the  structure of the borders should be identical for border collapse and separate but they are not, see the additional vertical line in row 2. This makes also doubt that your WFM mark in the other bug is correct.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Sorry.

But your comment 0 didn't say so. I think that your comment 0's bug is gone by another bug fix. However, there is still existing *another* problem, right?
And I don't think that this summary is good for actual problem.
Status: REOPENED → NEW
Created attachment 209271 [details]
testcase

If the table has border attribute, the non-existing cells have border. But if the table doesn't have border attribute, it's rendered correct.
Attachment #112619 - Attachment is obsolete: true
Attachment #133798 - Attachment is obsolete: true
Attachment #189136 - Attachment is obsolete: true
Summary: border-collapse problems at cellmap holes → border of non-existing cells are rendered if border-collapse table has border attribute
I think that this is *not* a bug.

The border of non-existing cells are not so. These borders are TR's border and COL's border. These borders are specified by table[rules="all"]. When I commented out as following, attachment 209271 [details]'s both tables are same result.

http://lxr.mozilla.org/mozilla/source/layout/style/nsHTMLStyleSheet.cpp#254
> 254 static void ColPostResolveCallback(nsStyleStruct* aStyleStruct, ... 
> 255 {
      #if 0
> 256   ::ProcessTableRulesAttribute(aStyleStruct, aRuleData, ...
> 257                                NS_STYLE_TABLE_RULES_COLS, ...
> 258   ::ProcessTableRulesAttribute(aStyleStruct, aRuleData, ...
> 259                                NS_STYLE_TABLE_RULES_COLS, ... 
      #endif
> 260 }
Created attachment 209273 [details]
testcase2

See this testcase. the right border of second cell of second row is gray or red. So, this border is column border. And the right border of third cell of second row is pink. So, this border is row border. If these borders are cell's border, they should be blue.
Whiteboard: INVA?
(Reporter)

Comment 12

12 years ago
Please either a) fix the bug and then do whatever you want with the description and feel free to invalidate my testcases, I am more than happy to review it or b) don't mess around with it. The description is a technical correct description of the issue. The description is intentional technical to make the bug usefull for people who are are able to fullfill choice a). See http://www.mozilla.org/newlayout/doc/table-cellmap.html what is a cellmap hole.
Summary: border of non-existing cells are rendered if border-collapse table has border attribute → border-collapse problems at cellmap holes
Whiteboard: INVA?
(Reporter)

Updated

9 years ago
Summary: border-collapse problems at cellmap holes → [BC] border-collapse problems at cellmap holes
Assignee: layout.tables → nobody
QA Contact: amar → layout.tables
(Reporter)

Updated

8 years ago
Depends on: 378935
(Reporter)

Comment 13

8 years ago
fixed by bug 378935 the testcase there also cover this bug
Flags: in-testsuite+
(Reporter)

Updated

8 years ago
Status: NEW → RESOLVED
Last Resolved: 12 years ago8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.