Bidi: Pressing "Home" or "End" on a blank line in a textarea in a bidi page moves the caret to the top of the textarea

RESOLVED FIXED in mozilla1.9alpha1

Status

()

RESOLVED FIXED
13 years ago
10 years ago

People

(Reporter: uriber, Assigned: uriber)

Tracking

({regression})

Trunk
mozilla1.9alpha1
regression
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

13 years ago
In any textarea (RTL or LTR), in a page containing RTL text (e.g., any of the textareas in attachment 128680 [details]):
1. Place the caret in an empty line
2. Press "Home" or "End" (Command+left/right-arrow on Mac).

Result: the caret jumps to the very top of the textarea. 
Expected: the caret shouldn't move.

This is a regression from bug 313602.
(Assignee)

Comment 1

13 years ago
Created attachment 213258 [details] [diff] [review]
patch

This restores the condition removed in bug 313602, but only for the Home/End case, thereby fixing this bug without regressing bug 313602 or bug 313596.
Attachment #213258 - Flags: review?(smontagu)
(Assignee)

Updated

13 years ago
Status: NEW → ASSIGNED

Updated

13 years ago
Attachment #213258 - Flags: review?(smontagu) → review+
(Assignee)

Updated

13 years ago
Attachment #213258 - Flags: superreview?(roc)
Attachment #213258 - Flags: superreview?(roc) → superreview+
(Assignee)

Comment 2

13 years ago
Checked in. Also s/cursor/caret/ in the comment.

Checking in layout/generic/nsSelection.cpp;
/cvsroot/mozilla/layout/generic/nsSelection.cpp,v  <--  nsSelection.cpp
new revision: 3.216; previous revision: 3.215
done
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9alpha

Updated

10 years ago
Component: Layout: BiDi Hebrew & Arabic → Layout: Text
QA Contact: zach → layout.fonts-and-text
You need to log in before you can comment on or make changes to this bug.