Closed
Bug 41959
Opened 24 years ago
Closed 24 years ago
missing background on blocks with only ' '
Categories
(Core :: Layout: Tables, defect, P3)
Core
Layout: Tables
Tracking
()
VERIFIED
FIXED
People
(Reporter: timeless, Assigned: shanjian)
References
()
Details
(Keywords: regression, testcase)
Attachments
(6 files)
The background from the css is ignored.
The CSS file will be attached to the blocked bug.
Blocks: 41957
In what build? It works for me in Linux mozilla 2000-06-08-08-M17.
Comment 4•24 years ago
|
||
The css file (http://sucs.swan.ac.uk/~firefury/beta/css/main.css) is no longer available. Closed as Invalid. You may want to attach the css file to the bug report and reopen.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → INVALID
Comment 5•24 years ago
|
||
The CSS file is also available here: http://bugzilla.mozilla.org/showattachment.cgi?attach_id=10056 I haven't checked that the bug still occurs. Marking qawanted to get some help. Whoever verifies this bug must make sure that the bug no longer occurs, by reconstructing the testcase from the HTML file attached to this bug and the CSS file given above.
Keywords: qawanted
Sorry, css file is now up.
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Pierre Saslawsky, next time please READ The comments and possibly follow the blocks, I just wasted space reuploading a file that was already on the server in the blocked bug. Ian Hickson thanks, but you gave me the html page instead of the css/bug number :-( oh well... I should have paid more attention.
Reporter | ||
Comment 10•24 years ago
|
||
As of 2000061518 <TD CLASS="leftborder"> </TD> <TD CLASS="rightborder"> </TD> do not receive the css background attribute. As for the test case I don't see the text it seems to be shreaded [I think I will give this a separate bug to parser].
Comment 11•24 years ago
|
||
The page at the URL above and the attached testcase work for me too on a 2000-06- 13 build on the Mac. Dbaron and rickg confirmed that it works on Unix and Windows too. Closed as WorksForMe. If you feel that this bug was closed in error, please provide your build ID and reopen.
Status: REOPENED → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → WORKSFORME
Reporter | ||
Comment 12•24 years ago
|
||
Excuse me, I did provide a build, and it's newer than yours. I'm sorry if my test cases don't validate. I'll have a better one when moz stops crashing on open dialog or supports drop target. Please attach a screen capture and circle all correctly rendered backgrounds, I'd like to see your works for me.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Comment 13•24 years ago
|
||
This bug is becoming a mess... Chris, you can ignore everything above. Timeless: sorry, I should have seen your comments. On your side, next time you file a bug please follow the bug writing guidelines on http://www.mozilla.org/ quality/bug-writing-guidelines.html. You can find some sample bug reports there too. The background doesn't show inside the 'leftborder' and 'rightborder' table cells because they contain only a ' ' so I guess they are considered empty. If you put some text in there, it works. I'm going to attach a testcase. Reassigned to Chris Karnaze/HTMLTables.
Assignee: pierre → karnaze
Status: REOPENED → NEW
Component: Style System → HTMLTables
Keywords: qawanted
QA Contact: ckritzer → desale
Summary: CSS Background ignored → missing background on ' ' cells
Comment 14•24 years ago
|
||
Comment 15•24 years ago
|
||
Updated•24 years ago
|
OS: Windows 2000 → All
Hardware: PC → All
Comment 16•24 years ago
|
||
Harish, we need to distinguish between and other whitespace. Nav considers a table cell with the former to not be empty. Can you investigate and see where the information is being lost (is it the parser, content sink or elsewhere). Nav gives the simple example a background in the 2nd cell, Mozilla does not. <head> <style> td { background-color: orange; } </style> </head> <table border> <tr> <td>foo</td> <td> </td> </tr> </table>
Assignee: karnaze → harishd
Comment 17•24 years ago
|
||
Chris's testcase seems to render correctly. Talked to Chris about it and he is willing to take the bug back :) Over to karnaze@netscape.com
Assignee: harishd → karnaze
Comment 18•24 years ago
|
||
*** Bug 47600 has been marked as a duplicate of this bug. ***
Comment 19•24 years ago
|
||
Hell, what's going on with this bug. It seemed to be ok, reoccured in the last first August, worked two days ago and is broken again with 2000080820 ?! I'm attaching the testcase of #47600.
Comment 20•24 years ago
|
||
Comment 21•24 years ago
|
||
Weird... Reassigned to attinasi who is covering Tables for karnaze.
Assignee: karnaze → attinasi
Comment 22•24 years ago
|
||
I confirm the bug, and it is weird. My guess is it is something in line layout (or text frame) since no table code has been changing, and the style context data is all correct. Note also that this is working fine in M17 (PR2).
Status: NEW → ASSIGNED
Comment 23•24 years ago
|
||
This looks like a special case (dup) of bug 45562. In both cases, the addition of a newline in front of or behind the makes it work. Also, this problem is not restriceted to tables, it is also happening on DIVs (testcase to follow). Marking this a dup of 45562 and updating 45562 to note this special case issue and the testcases to include in verification. Note: it is unclear why this bug depends on bug 42781. *** This bug has been marked as a duplicate of 45562 ***
Status: ASSIGNED → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → DUPLICATE
Summary: missing background on ' ' cells → missing background on blocks with only ' '
Comment 24•24 years ago
|
||
I think you got the wrong bug number. I don't see how this is a duplicate of a bug on text widgets and incremental reflow. Reopening.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Comment 26•24 years ago
|
||
That was the correct bug number, and maybe it is not a dup. I think bug 45562 has a couple of issues. The part I noticed was how the whitespace in the markup was affecting the layout (see bug 45562, comment by rods@netscape.com 2000-08-09 09:37). So I'll leave this open and just note that there are some similarities between this bug and a comment in that bug. Thanks, David.
Status: REOPENED → ASSIGNED
Comment 27•24 years ago
|
||
shanjian made a pair of changes on 8/3 for bug 14588 that broke behavior a bit. http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&fi le=nsTextTransformer.cpp&root=/cvsroot&subdir=mozilla/layout/html/base/src&comma nd=DIFF_FRAMESET&rev1=1.39&rev2=1.40 and http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&fi le=nsTextFrame.cpp&root=/cvsroot&subdir=mozilla/layout/html/base/src&command=DIF F_FRAMESET&rev1=1.266&rev2=1.267 Backing out these two changes fixes this bug. Over to you, shanjian :)
Comment 28•24 years ago
|
||
*** Bug 48554 has been marked as a duplicate of this bug. ***
Comment 29•24 years ago
|
||
*** Bug 48654 has been marked as a duplicate of this bug. ***
Comment 30•24 years ago
|
||
*** Bug 48131 has been marked as a duplicate of this bug. ***
Comment 31•24 years ago
|
||
*** Bug 48569 has been marked as a duplicate of this bug. ***
Comment 32•24 years ago
|
||
Fixed when nsTextTransformer.cpp was backed out to ver. 1.39
Status: NEW → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → FIXED
Comment 33•24 years ago
|
||
*** Bug 48984 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•