Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Fix down arrow in AutoCompleteTextView." into honeycomb

Browse Source
This commit is contained in:
Jeff Brown
2011-01-26 23:45:59 -08:00
committed by Android (Google) Code Review
parent d529d0df8f 8d6d3b83fb
commit f02853f211
4 changed files with 96 additions and 80 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
core/java/android/widget/AbsListView.java
View File

@@ -4467,15 +4467,14 @@ public abstract class AbsListView extends AdapterView<ListAdapter> implements Te
}
/**
* If there is a selection returns true.
* Otherwise resurrects the selection and returns false.
* If there is a selection returns false.
* Otherwise resurrects the selection and returns true if resurrected.
*/
boolean ensureSelectionOnMovementKey() {
boolean resurrectSelectionIfNeeded() {
if (mSelectedPosition < 0) {
resurrectSelection();
return false;
return resurrectSelection();
}
return true;
return false;
}
/**

80
core/java/android/widget/GridView.java
View File

@@ -1486,77 +1486,79 @@ public class GridView extends AbsListView {
switch (keyCode) {
case KeyEvent.KEYCODE_DPAD_LEFT:
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && arrowScroll(FOCUS_LEFT);
handled = resurrectSelectionIfNeeded() || arrowScroll(FOCUS_LEFT);
}
break;
case KeyEvent.KEYCODE_DPAD_RIGHT:
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && arrowScroll(FOCUS_RIGHT);
handled = resurrectSelectionIfNeeded() || arrowScroll(FOCUS_RIGHT);
}
break;
case KeyEvent.KEYCODE_DPAD_UP:
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && arrowScroll(FOCUS_UP);
handled = resurrectSelectionIfNeeded() || arrowScroll(FOCUS_UP);
} else if (event.hasModifiers(KeyEvent.META_ALT_ON)) {
handled = ensureSelectionOnMovementKey() && fullScroll(FOCUS_UP);
handled = resurrectSelectionIfNeeded() || fullScroll(FOCUS_UP);
}
break;
case KeyEvent.KEYCODE_DPAD_DOWN:
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && arrowScroll(FOCUS_DOWN);
handled = resurrectSelectionIfNeeded() || arrowScroll(FOCUS_DOWN);
} else if (event.hasModifiers(KeyEvent.META_ALT_ON)) {
handled = ensureSelectionOnMovementKey() && fullScroll(FOCUS_DOWN);
handled = resurrectSelectionIfNeeded() || fullScroll(FOCUS_DOWN);
}
break;
case KeyEvent.KEYCODE_DPAD_CENTER:
case KeyEvent.KEYCODE_ENTER: {
if (event.hasNoModifiers()
&& event.getRepeatCount() == 0 && getChildCount() > 0) {
ensureSelectionOnMovementKey();
keyPressed();
case KeyEvent.KEYCODE_ENTER:
if (event.hasNoModifiers()) {
handled = resurrectSelectionIfNeeded();
if (!handled
&& event.getRepeatCount() == 0 && getChildCount() > 0) {
keyPressed();
handled = true;
}
}
return true;
}
break;
case KeyEvent.KEYCODE_SPACE:
if (mPopup == null || !mPopup.isShowing()) {
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && pageScroll(FOCUS_DOWN);
handled = resurrectSelectionIfNeeded() || pageScroll(FOCUS_DOWN);
} else if (event.hasModifiers(KeyEvent.META_SHIFT_ON)) {
handled = ensureSelectionOnMovementKey() && pageScroll(FOCUS_UP);
handled = resurrectSelectionIfNeeded() || pageScroll(FOCUS_UP);
}
}
break;
case KeyEvent.KEYCODE_PAGE_UP:
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && pageScroll(FOCUS_UP);
handled = resurrectSelectionIfNeeded() || pageScroll(FOCUS_UP);
} else if (event.hasModifiers(KeyEvent.META_ALT_ON)) {
handled = ensureSelectionOnMovementKey() && fullScroll(FOCUS_UP);
handled = resurrectSelectionIfNeeded() || fullScroll(FOCUS_UP);
}
break;
case KeyEvent.KEYCODE_PAGE_DOWN:
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && pageScroll(FOCUS_DOWN);
handled = resurrectSelectionIfNeeded() || pageScroll(FOCUS_DOWN);
} else if (event.hasModifiers(KeyEvent.META_ALT_ON)) {
handled = ensureSelectionOnMovementKey() && fullScroll(FOCUS_DOWN);
handled = resurrectSelectionIfNeeded() || fullScroll(FOCUS_DOWN);
}
break;
case KeyEvent.KEYCODE_MOVE_HOME:
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && fullScroll(FOCUS_UP);
handled = resurrectSelectionIfNeeded() || fullScroll(FOCUS_UP);
}
break;
case KeyEvent.KEYCODE_MOVE_END:
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && fullScroll(FOCUS_DOWN);
handled = resurrectSelectionIfNeeded() || fullScroll(FOCUS_DOWN);
}
break;
@@ -1569,32 +1571,34 @@ public class GridView extends AbsListView {
// perhaps it should be configurable (and more comprehensive).
if (false) {
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && sequenceScroll(FOCUS_FORWARD);
handled = resurrectSelectionIfNeeded()
|| sequenceScroll(FOCUS_FORWARD);
} else if (event.hasModifiers(KeyEvent.META_SHIFT_ON)) {
handled = ensureSelectionOnMovementKey() && sequenceScroll(FOCUS_BACKWARD);
handled = resurrectSelectionIfNeeded()
|| sequenceScroll(FOCUS_BACKWARD);
}
}
break;
}
}
if (!handled) {
handled = sendToTextFilter(keyCode, count, event);
}
if (handled) {
return true;
} else {
switch (action) {
case KeyEvent.ACTION_DOWN:
return super.onKeyDown(keyCode, event);
case KeyEvent.ACTION_UP:
return super.onKeyUp(keyCode, event);
case KeyEvent.ACTION_MULTIPLE:
return super.onKeyMultiple(keyCode, count, event);
default:
return false;
}
}
if (sendToTextFilter(keyCode, count, event)) {
return true;
}
switch (action) {
case KeyEvent.ACTION_DOWN:
return super.onKeyDown(keyCode, event);
case KeyEvent.ACTION_UP:
return super.onKeyUp(keyCode, event);
case KeyEvent.ACTION_MULTIPLE:
return super.onKeyMultiple(keyCode, count, event);
default:
return false;
}
}

2
core/java/android/widget/ListPopupWindow.java
View File

@@ -774,7 +774,7 @@ public class ListPopupWindow {
}
/**
* Filter key down events. By forwarding key up events to this function,
* Filter key down events. By forwarding key down events to this function,
* views using non-modal ListPopupWindow can have it handle key selection of items.
*
* @param keyCode keyCode param passed to the host view's onKeyDown

83
core/java/android/widget/ListView.java
View File

@@ -2081,25 +2081,35 @@ public class ListView extends AbsListView {
switch (keyCode) {
case KeyEvent.KEYCODE_DPAD_UP:
if (event.hasNoModifiers()) {
if (ensureSelectionOnMovementKey()) {
handled = resurrectSelectionIfNeeded();
if (!handled) {
while (count-- > 0) {
handled |= arrowScroll(FOCUS_UP);
if (arrowScroll(FOCUS_UP)) {
handled = true;
} else {
break;
}
}
}
} else if (event.hasModifiers(KeyEvent.META_ALT_ON)) {
handled = ensureSelectionOnMovementKey() && fullScroll(FOCUS_UP);
handled = resurrectSelectionIfNeeded() || fullScroll(FOCUS_UP);
}
break;
case KeyEvent.KEYCODE_DPAD_DOWN:
if (event.hasNoModifiers()) {
if (ensureSelectionOnMovementKey()) {
handled = resurrectSelectionIfNeeded();
if (!handled) {
while (count-- > 0) {
handled |= arrowScroll(FOCUS_DOWN);
if (arrowScroll(FOCUS_DOWN)) {
handled = true;
} else {
break;
}
}
}
} else if (event.hasModifiers(KeyEvent.META_ALT_ON)) {
handled = ensureSelectionOnMovementKey() && fullScroll(FOCUS_DOWN);
handled = resurrectSelectionIfNeeded() || fullScroll(FOCUS_DOWN);
}
break;
@@ -2117,19 +2127,22 @@ public class ListView extends AbsListView {
case KeyEvent.KEYCODE_DPAD_CENTER:
case KeyEvent.KEYCODE_ENTER:
if (mItemCount > 0 && event.getRepeatCount() == 0) {
ensureSelectionOnMovementKey();
keyPressed();
if (event.hasNoModifiers()) {
handled = resurrectSelectionIfNeeded();
if (!handled
&& event.getRepeatCount() == 0 && getChildCount() > 0) {
keyPressed();
handled = true;
}
}
handled = true;
break;
case KeyEvent.KEYCODE_SPACE:
if (mPopup == null || !mPopup.isShowing()) {
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && pageScroll(FOCUS_DOWN);
handled = resurrectSelectionIfNeeded() || pageScroll(FOCUS_DOWN);
} else if (event.hasModifiers(KeyEvent.META_SHIFT_ON)) {
handled = ensureSelectionOnMovementKey() && pageScroll(FOCUS_UP);
handled = resurrectSelectionIfNeeded() || pageScroll(FOCUS_UP);
}
handled = true;
}
@@ -2137,29 +2150,29 @@ public class ListView extends AbsListView {
case KeyEvent.KEYCODE_PAGE_UP:
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && pageScroll(FOCUS_UP);
handled = resurrectSelectionIfNeeded() || pageScroll(FOCUS_UP);
} else if (event.hasModifiers(KeyEvent.META_ALT_ON)) {
handled = ensureSelectionOnMovementKey() && fullScroll(FOCUS_UP);
handled = resurrectSelectionIfNeeded() || fullScroll(FOCUS_UP);
}
break;
case KeyEvent.KEYCODE_PAGE_DOWN:
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && pageScroll(FOCUS_DOWN);
handled = resurrectSelectionIfNeeded() || pageScroll(FOCUS_DOWN);
} else if (event.hasModifiers(KeyEvent.META_ALT_ON)) {
handled = ensureSelectionOnMovementKey() && fullScroll(FOCUS_DOWN);
handled = resurrectSelectionIfNeeded() || fullScroll(FOCUS_DOWN);
}
break;
case KeyEvent.KEYCODE_MOVE_HOME:
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && fullScroll(FOCUS_UP);
handled = resurrectSelectionIfNeeded() || fullScroll(FOCUS_UP);
}
break;
case KeyEvent.KEYCODE_MOVE_END:
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && fullScroll(FOCUS_DOWN);
handled = resurrectSelectionIfNeeded() || fullScroll(FOCUS_DOWN);
}
break;
@@ -2172,35 +2185,35 @@ public class ListView extends AbsListView {
// perhaps it should be configurable (and more comprehensive).
if (false) {
if (event.hasNoModifiers()) {
handled = ensureSelectionOnMovementKey() && arrowScroll(FOCUS_DOWN);
handled = resurrectSelectionIfNeeded() || arrowScroll(FOCUS_DOWN);
} else if (event.hasModifiers(KeyEvent.META_SHIFT_ON)) {
handled = ensureSelectionOnMovementKey() && arrowScroll(FOCUS_UP);
handled = resurrectSelectionIfNeeded() || arrowScroll(FOCUS_UP);
}
}
break;
}
}
if (!handled) {
handled = sendToTextFilter(keyCode, count, event);
}
if (handled) {
return true;
} else {
switch (action) {
case KeyEvent.ACTION_DOWN:
return super.onKeyDown(keyCode, event);
}
case KeyEvent.ACTION_UP:
return super.onKeyUp(keyCode, event);
if (sendToTextFilter(keyCode, count, event)) {
return true;
}
case KeyEvent.ACTION_MULTIPLE:
return super.onKeyMultiple(keyCode, count, event);
switch (action) {
case KeyEvent.ACTION_DOWN:
return super.onKeyDown(keyCode, event);
default: // shouldn't happen
return false;
}
case KeyEvent.ACTION_UP:
return super.onKeyUp(keyCode, event);
case KeyEvent.ACTION_MULTIPLE:
return super.onKeyMultiple(keyCode, count, event);
default: // shouldn't happen
return false;
}
}