Write a metrofx selection test for bug 865654

RESOLVED INCOMPLETE

Status

defect
RESOLVED INCOMPLETE
6 years ago
2 years ago

People

(Reporter: jimm, Unassigned)

Tracking

Trunk
x86_64
Windows 8.1
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [selection] p=0)

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
Posted file testcase
When we move text input caret position in response to touch input, we adjust touch coordinates up such that they are in the input vs below the lower boundary. However when dealing with text inputs that have a lot of white space, caretPositionFromPoint can return the wrong offsets for coordinates that land in the white space vs on text, messing up caret position.

We should do a better job of centering here somehow to avoid this.
(Reporter)

Updated

6 years ago
Blocks: 865356
(Reporter)

Comment 1

6 years ago
Updating this now that bug 865654 is fixed, the original problem reported is now WFM. We should add a test using this test case so that we can catch future breakage.
Component: Input → Tests
Summary: SelectionHandler should center _onCaretMove coordinates to compensate for bug 865654 → Write a metrofx selection test for bug 865654
(Reporter)

Updated

5 years ago
Blocks: 957244
Blocks: metrobacklog
Whiteboard: [selection] → [selection] p=0
OS: Windows 8 Metro → Windows 8.1
Mass close of bugs in obsolete product https://bugzilla.mozilla.org/show_bug.cgi?id=1350354
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.