96354 - Editing tables with borders inside of tables without borders

Status: VERIFIED FIXED

(SeaMonkey :: Composer, defect)

Description

[TucsonTester2]

From Bugzilla Helper:
User-Agent: Mozilla/4.77 [en] (Win98; U)
BuildID:    20010816

If you insert a table with a value of zero for the border into the into the 
composer and then insert a table into one of the cells and make that table have 
a border, the editor will show the table with a border as a dotted border as if 
there was no border at all.  There is a small line that does show some of the 
inner tables border though.

Reproducible: Always
Steps to Reproduce:
1.open composer
2.click on the table button on the toolbar and make a default table (2x2 100% 
width)
3.double click on the table and click on the table tab, then make the border 
value 0
4.click in any one of the cells in the table to place the blinking cursor inside
5. click on the insert menu and choose table, just use the default table again.

Actual Results:  The table inside of the table appears to have half a border and 
half of a border that should be invisible.  If you switch to preview mode the 
table appears properly.  Back in the normal editor mode the table is still 
showing improperly though.

Expected Results:  I expect to see clearly that the inner table has a border.  
It just helps to see this to know how my page is laid out.

Comment 1

[Kathleen :Brade]

-->cmanske

Comment 2

[Greg K.]

Confirming using Mac/2001090508.

Comment 3

[Charles Manske]

In latest debug build, the inner table does show the border correctly.
But the cells in that table should also show the border, as they do in Preview
mode. The inner table's cells are showing the dotted border as if they were
in a table with border="0", however. This is a Composer CSS bug.
Daniel: It seems that this part of the CSS rule:
  table[border="0"] td, table[border="0"] th,
...
{
  border: 1px dotted red;
}
causes the inner-most cells to get the dotted line, even though the immediate
parent table has a border > 0, right?
Can you think of a CSS rule to correct this without getting into complicated
">" notation for child cells?

Comment 4

[Syd Logan]

spam composer change

Comment 5

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

getting ownership of this bug since I am the original author of the css rule
shown above. tagging EDITORBASE.

Comment 6

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

Attachment: patch v1.0 ; ready for reviews

Comment 7

[jag (Peter Annema)]

Comment on attachment 50674 [details] [diff] [review]
patch v1.0 ; ready for reviews

r=jag

Comment 8

[kinmoz]

Comment on attachment 50674 [details] [diff] [review]
patch v1.0 ; ready for reviews

sr=[e-mail]

Comment 9

[Daniel Glazman (:glazou) (not active in Mozilla any more)]

checked in into trunk.

Comment 10

[James Baxter]

Verified in 10-01 Trunk Build