49392 - Editor inserts unneeded <br> tags in table cells

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect, P3)

Description

[Akkana Peck]

Start with a file that has a very simple table in it (see attachment).  Load it
into the browser -- looks fine.  Now load it into the editor -- there are br
tags where there shouldn't be, making the table look wrong.

We shouldn't insert br tags on non-blank table cells.

Comment 1

[Akkana Peck]

Nom. for nsbeta3 -- can't use composer to edit existing files if it screws up
all the table cells.  I can't even use it for testing my output bugs, since the
extra br's are throwing off the line breaks issues I'm trying to fix.

Comment 2

[Akkana Peck]

Attachment: Very simple table test

Comment 3

[Joe Francis]

ok, so the stuff mike stayed up all night writing for me at the end of beta2 is 
not working right: it's telling me that the table cells dont have anything 
actually rendering in them.  It turns out that it was all a bad idea to start 
with, so I'm not going to ask him to fix it.  Instead I'll just stop using it, 
and give up on getting the info I need from layout, since layout basically has no 
way to know without doing a reflow.  

So it's fixed.  The flip side is that this will cause another bug to reopen: if 
you have that same table but put in a single space instead of a single number in 
each cell, you now WONT get the brs that you DO need.

But I'm gonna have to come up with another way to solve that.

Comment 4

[sujay]

okay br tags are gone for non-blank table cells

verified in 10/30 build.