Closed
Bug 89452
Opened 24 years ago
Closed 24 years ago
Insert Table Row in multi row/col table inserts row with text
Categories
(Core :: DOM: Editor, defect, P3)
Core
DOM: Editor
Tracking
()
VERIFIED
FIXED
mozilla0.9.4
People
(Reporter: sujay, Assigned: karnaze)
References
Details
(Whiteboard: [QAHP])
Attachments
(2 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
using 7/5 build of netscape:
1) launch netscape
2) launch composer
3) insert 2 x 2 table
4) populate each cell with text
5) click in a cell in top row
6) Table Insert | Row Below
notice the row gets added, but that row has text in it!
it should be blank.
WORKAROUND = minimize window and then maximize window
then you have a blank/empty row
this is a repaint/layout issue
Comment 4•24 years ago
|
||
Well, we can try for 0.9.3 - I wonder if the recent fix (on the trunk) for bug
86852 fixes this? The fix was to correct an optimization that made too small of
an area invalidate when the content was changed. Sujay, can you check it out on
a new trunk build? Thanks.
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → mozilla0.9.3
Marc, I just downloaded 7/10 trunk build.
same problem..
it adds a new Row Below, but it has text in it.
only after you force a repaint(minimize followed by maximize),
then the Row have blank cells.
Comment 7•24 years ago
|
||
Still seeing this using 2001-07-20-03 under Win2k-sp1. It is only a redraw
problem, or so it seems. I can't "edit" the text in the new row, because its not
really there. If you save right after adding the row, it saves without any text.
Quick fix could just be to force a redraw after every table configuration change
(insert or delete AND row or column) since there is no data loss, just an annoyance.
Comment 8•24 years ago
|
||
I think I was suppossed to send this to karnaze last week. Over to Chris now. If
you want me to check this in block-land, jsut send it back, but it sounds like a
table-specific problem.
Comment 9•24 years ago
|
||
ok, really reassigning this time
Assignee: attinasi → karnaze
Status: ASSIGNED → NEW
Assignee | ||
Comment 12•24 years ago
|
||
Comment 13•24 years ago
|
||
Couple of comments on the patch:
1) if you use an nsAutoVoidArray instead of an nsVoidArray you might save on
some mallocs. I'm not sure how many rows you will be putting in there, but it
might be worth checking out
2) You should assert (or check) rgFrame before you deref (in case the ElementAt
returns you null).
3) After you do the invalidation, don't you want to break out of the loops and
stop searching the rows and rowgroups?
Consider / answer these potential issues, and [s]r=attinasi
Comment 14•24 years ago
|
||
Comment 15•24 years ago
|
||
r=alexsavulov
Assignee | ||
Comment 16•24 years ago
|
||
Assignee | ||
Comment 17•24 years ago
|
||
The patch is in.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Comment 18•24 years ago
|
||
*** Bug 88976 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
•