diff options
author | Gilles Debunne <debunne@google.com> | 2010-08-23 15:14:03 -0700 |
---|---|---|
committer | Gilles Debunne <debunne@google.com> | 2010-08-24 10:18:31 -0700 |
commit | 2703a42d16af0e62da1bba02b6c935d98debf936 (patch) | |
tree | d8c04e0e9080eb2e06d8ddd7f7dd77707dd4dcc1 /core/java/android/text/method | |
parent | b6d71351c074d5c0bc13a91544d776f1524eaabd (diff) | |
download | frameworks_base-2703a42d16af0e62da1bba02b6c935d98debf936.zip frameworks_base-2703a42d16af0e62da1bba02b6c935d98debf936.tar.gz frameworks_base-2703a42d16af0e62da1bba02b6c935d98debf936.tar.bz2 |
When an EditText gains focus by tapping, move the insertion point where tapped.
The previous behavior was to move cursor at the end of the first line,
which feels weird.
Change-Id: I5a72f9871ed79ee2c521698ea642ba126537f4f9
Diffstat (limited to 'core/java/android/text/method')
-rw-r--r-- | core/java/android/text/method/ArrowKeyMovementMethod.java | 20 |
1 files changed, 2 insertions, 18 deletions
diff --git a/core/java/android/text/method/ArrowKeyMovementMethod.java b/core/java/android/text/method/ArrowKeyMovementMethod.java index baaf39d..3fe14f9 100644 --- a/core/java/android/text/method/ArrowKeyMovementMethod.java +++ b/core/java/android/text/method/ArrowKeyMovementMethod.java @@ -319,25 +319,9 @@ public class ArrowKeyMovementMethod implements MovementMethod { public void onTakeFocus(TextView view, Spannable text, int dir) { if ((dir & (View.FOCUS_FORWARD | View.FOCUS_DOWN)) != 0) { - Layout layout = view.getLayout(); - - if (layout == null) { - /* - * This shouldn't be null, but do something sensible if it is. - */ + if (view.getLayout() == null) { + // This shouldn't be null, but do something sensible if it is. Selection.setSelection(text, text.length()); - } else { - /* - * Put the cursor at the end of the first line, which is - * either the last offset if there is only one line, or the - * offset before the first character of the second line - * if there is more than one line. - */ - if (layout.getLineCount() == 1) { - Selection.setSelection(text, text.length()); - } else { - Selection.setSelection(text, layout.getLineStart(1) - 1); - } } } else { Selection.setSelection(text, text.length()); |