16481 - [DOGFOOD]Deleting text in table deletes table cells

Status: VERIFIED FIXED

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

Description

[Charles Manske]

When using the delete key or backspace to delete contents in a table cell,
after the last text character or element is deleted, the table cell is deleted.
It should not. It should skip over to the contents of the previous cell when
using backspace, or just stop deleting when using the delete key.
Otherwise holding down delete or backspace really messes up your table
structure!

Comment 1

[Joe Francis]

accepting/m11

Comment 2

[rubydoo123]

submitting this as a dogfood candidate -- deleting the actual cell is not a good
thing, it will actually affect the entire layout of the page in many instances.

Comment 3

[leger]

Can you recreate your cell?

Comment 4

[rubydoo123]

this will need a deletion rules for table elements, this is going to take some
time to do. Joe knows what to do but it will not be done by today. Moving over
to M12

Comment 5

[leger]

If there is a workaround, let's make this PDT- and put workaround into:
http://bugzilla.mozilla.org/show_bug.cgi?id=17788

Otherwise, yes fix for dogfood.

Comment 6

[rubydoo123]

linking to PDT+ tracking bug 12658

Comment 7

[rubydoo123]

this one has been investigated, joe has a good idea on how to fix it

Comment 8

[Joe Francis]

In progress.  revising estimate to 12/3

Comment 9

[Joe Francis]

I'm finally starting this.  Still estimate a 12/3 resolution.

Comment 10

[Joe Francis]

fixed.  should make this mornings verification builds.

The behvior now is:  If you have an insertion point and hit delete or forward
delete, you will stop when you get to the boundary of any table element.  If you
have a selection range highlighted and hit delete or forward delete, nothing will
happen if your selection endpoints reside in different table elements (including
one in a table element and one not).

Note that the various insert operations will implicitly delete a selection range.
These actions do not yet play by these deletion rules.  That's a sperate bug.  I
need to make sure that everything that implicitly deletes, does so via the rules
code so that we get a chance to cancel illegal table deletions.

Comment 11

[sujay]

verified in 12/1 build.