Last Comment Bug 72359 - Anonymous table cells get extra margins?
: Anonymous table cells get extra margins?
[Hixie-P4] [have fix]
: css2, testcase
Product: Core
Classification: Components
Component: Layout: Tables (show other bugs)
: Trunk
: All All
-- minor (vote)
: mozilla1.0.1
Assigned To: Hixie (not reading bugmail)
: Amarendra Hanumanula
Depends on: 72358 78695
  Show dependency treegraph
Reported: 2001-03-17 14:43 PST by Hixie (not reading bugmail)
Modified: 2014-04-25 15:15 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (414 bytes, patch)
2003-05-27 22:53 PDT, Bernd
bernd_mozilla: review+
dbaron: superreview+
Details | Diff | Splinter Review

Description User image Hixie (not reading bugmail) 2001-03-17 14:43:36 PST
It appears that the anonymous cell marked in the fragment below with "***" is
getting an extra margin.

  <float xmlns="">
     <!--cell-->   <!-- *** -->
        <cell>SHOULD B</cell>
        <cell>E N</cell>
        <cell>O RED</cell>

See for a test case.
Note that until bug 72358 is fixed it will be hard to see this bug.
Comment 1 User image karnaze (gone) 2001-03-17 18:02:50 PST
Moving to m1.0
Comment 2 User image Amarendra Hanumanula 2001-03-22 13:12:27 PST
QA contact update
Comment 3 User image Hixie (not reading bugmail) 2001-05-21 15:45:51 PDT
Hyatt has a fix for this on his branch.
Comment 4 User image Peter Trudelle 2001-06-01 16:59:56 PDT
not fixed on branch, ->1.0
Comment 5 User image Hixie (not reading bugmail) 2001-06-22 03:59:12 PDT
Incidentally, the fix is simply to remove the padding on :table-cell and move it
to the <td> rule. We just need the table folk to agree to it. Karnaze?
Comment 6 User image basic 2001-08-15 00:29:21 PDT
what's the status on this?
Comment 7 User image basic 2002-02-20 17:44:45 PST
Comment 8 User image Hixie (not reading bugmail) 2002-02-21 00:12:20 PST
Taking bug
Comment 9 User image karnaze (gone) 2002-02-21 07:18:32 PST
ian, if you make this change, then you need to run the block and table 
regression tests.
Comment 10 User image Hixie (not reading bugmail) 2002-02-21 07:30:36 PST
you bet
Comment 11 User image Greg K. 2002-07-14 01:07:36 PDT
Reconfirmed using FizzillaCFM/2002071208. Setting All/All.
Comment 12 User image Bernd 2003-05-27 22:53:39 PDT
Created attachment 124334 [details] [diff] [review]

we can safely remove this from ua.css as it is duplicated at html.css

I runned the regression tests:
the tests that failed are:

 and the 2479-2 ...

the tests 2479-xx are made by hixie , so the probability that one will ever see
them in normal life is zero.
Comment 13 User image Bernd 2003-05-31 02:30:47 PDT
fix + testcase checked in

Note You need to log in before you can comment on or make changes to this bug.