363378 - Caret gets stuck (or moves incorrectly) in empty table cells when using left/right arrow keys

Status: RESOLVED FIXED

(Core :: DOM: Selection, defect)

Description

[Uri Bernstein]

In caret browsing mode, using the the left/right arrow keys to navigate into an empty table cell may result in the caret disappearing from the screen, and becoming unmovable by further left/right arrow presses.

The description above is true when there is whitespace in the HTML around the empty <td> element. if there is no whitespace, the caret doesn't disappear into limbo, but still moves incorrectly and inconsistently.

Similar problems may occur with other kinds of empty block container frames (e.g., empty divs).

This is a regression from the "phase 1" patch to bug 300131.

Testcase and patch coming up.

Comment 1

[Uri Bernstein]

Attachment: testcase

Instructions included.

Comment 2

[Uri Bernstein]

Attachment: patch v1

Don't allow the caret to rest in (empty) container frames.

Comment 3

[Uri Bernstein]

Checked in:
Checking in mozilla/layout/generic/nsContainerFrame.cpp;
/cvsroot/mozilla/layout/generic/nsContainerFrame.cpp,v  <--  nsContainerFrame.cpp
new revision: 1.264; previous revision: 1.263
done
Checking in mozilla/layout/generic/nsContainerFrame.h;
/cvsroot/mozilla/layout/generic/nsContainerFrame.h,v  <--  nsContainerFrame.h
new revision: 3.73; previous revision: 3.72
done