45281 - Cannot insert a table caption

Status: VERIFIED FIXED

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

Description

[sujay]

using 7/12 build of netscape

1) launch netscape
2) launch composer
3) insert default table
4) go into Table Props
5) click on caption

try entering a caption either above/below the table(as you chose in props)

it doesn't work

all platforms.

Comment 1

[Charles Manske]

The node insertion code isn't allowing use to insert a <caption> element.

Comment 2

[rubydoo123]

setting to nsbeta3+
Charley, can you please add the level of priority -- thanks

Comment 3

[Charles Manske]

The problem is in the rules code. Since we create a new caption element with
no content, the rules code is automatically remove it. Joe needs to treat 
<caption> like table cells and automatically insert <br> and not delete the
element.

Comment 4

[Joe Francis]

charlie, i'll try to get you a patch for this today.

Comment 5

[Joe Francis]

Attachment: diffs for nsHTMLEditUtils.cpp

Comment 6

[Joe Francis]

attached a diff; charlie, give this a try

Comment 7

[Joe Francis]

ok, my diffs stank.  i sent charlie some new ones.

Comment 8

[Joe Francis]

fixed.  there is a different bug now: you cant click in the caption.  I filed 
48861 on this issue. 

sujay: to test create a table, then use the table dialog to create a caption.  
Look at the html source after that and you'll see an empty caption (with a <br> 
in it) in place.

Comment 9

[sujay]

still not fixed in 8/15 build....Charley and I just saw this one together...
I can't click inside a caption and enter text. 

this could be dependent on a bug that Joe filed to Mike....I don't
have that bug number.

Comment 10

[Joe Francis]

Sujay, please reread my earlier comment.  To verify this bug, look at the html 
source after inserting the caption and see that the caption is there, and has a <
br> in it.  This bug is fixed.

Not being able to click in it is a seperate bug against mike: 48861.

Comment 11

[sujay]

verified in 8/22 build.