Safari: Press delete button in an empty cell deleting empty Rows below current row

[satya]

To reproduce the defect:

1. Open any CK Editor sample and insert a table with default values(3 Rows & 2 Columns).

2. Don't enter any content in second or third rows.

3. Enter the content in first cell in first row, leave second cell in first row empty.

4. Keep cursor in the Second cell on first row & press Delete

Issue: Second row is deleted

5. Press Delete again..

Issue: Next empty row gets deleted

[j.swiderski]

Reproducible in Webkit from CKEditor 3.0

Notes:

• There doesn't need to be any text in first row. Just put cursor in first row in right-most cell and press delete.
• Similar to already closed #1722

[damo]

What is the plan to resolve this ticket?

[j.swiderski]

Just a guess but perhaps #8471 has the same cause for this behavior.

[j.swiderski]

What is the plan to resolve this ticket?

As discussed with senior developers - This one will be aimed for v4 only, it's just complex for this release.

[garry.yao]

Opened t/8916: Backspace/Del keystrokes are now handled at the boundary of table cells, to move cursor.

[Reinmar]

I pushed rebased branch + small coding style fix.

Del/backspace behaviour is fixed on Webkits.

In the code I see that this fix is generic and I wonder if it shouldn't affect also IEs? I also think that this change may deserve a new TC.

[fredck]

Making the change for all browsers makes sense, as it guarantees a unified behaviour among them, no matter their default behaviour. Considering that we're have that code for a single browser anyway, it is safe to just enlarge it to others.

I'm still not a big fan of making DEL a movement key. I prefer the IE behaviour for this TC, where nothing happens in this case.

Anyway, the proposed behaviour is "acceptable". Let's see if people will complain about it.

[garry.yao]

There's a problem with having test coverage on keystroke handlers in v4, see #9300.