Merge "Add visual dividers to more suggestions panel"

main
Tadashi G. Takaoka 2011-09-08 07:25:17 -07:00 committed by Android (Google) Code Review
commit a08a7e99b4
9 changed files with 48 additions and 16 deletions

Binary file not shown.

After

Width:  |  Height:  |  Size: 1.0 KiB

View File

Before

Width:  |  Height:  |  Size: 1012 B

After

Width:  |  Height:  |  Size: 1012 B

Binary file not shown.

After

Width:  |  Height:  |  Size: 2.9 KiB

View File

Before

Width:  |  Height:  |  Size: 2.7 KiB

After

Width:  |  Height:  |  Size: 2.7 KiB

Binary file not shown.

After

Width:  |  Height:  |  Size: 1.0 KiB

Binary file not shown.

After

Width:  |  Height:  |  Size: 1.0 KiB

View File

@ -22,6 +22,6 @@
xmlns:android="http://schemas.android.com/apk/res/android" xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="wrap_content" android:layout_width="wrap_content"
android:layout_height="match_parent" android:layout_height="match_parent"
android:src="@drawable/keyboard_suggest_strip_divider" android:src="@drawable/suggestions_strip_divider"
android:padding="0dp" android:padding="0dp"
android:gravity="center" /> android:gravity="center" />

View File

@ -573,6 +573,13 @@ public class Key {
super(res, params, row, parser, keyStyles); super(res, params, row, parser, keyStyles);
} }
/**
* This constructor is being used only for divider in more keys keyboard.
*/
public Spacer(KeyboardParams params, Drawable icon, int x, int y, int width, int height) {
super(params, null, null, icon, Keyboard.CODE_DUMMY, null, x, y, width, height, 0);
}
@Override @Override
public boolean isSpacer() { public boolean isSpacer() {
return true; return true;

View File

@ -16,7 +16,9 @@
package com.android.inputmethod.latin; package com.android.inputmethod.latin;
import android.content.res.Resources;
import android.graphics.Paint; import android.graphics.Paint;
import android.graphics.drawable.Drawable;
import android.text.TextUtils; import android.text.TextUtils;
import com.android.inputmethod.keyboard.Key; import com.android.inputmethod.keyboard.Key;
@ -49,14 +51,21 @@ public class MoreSuggestions extends Keyboard {
private final int[] mNumColumnsInRow = new int[SuggestionsView.MAX_SUGGESTIONS]; private final int[] mNumColumnsInRow = new int[SuggestionsView.MAX_SUGGESTIONS];
private static final int MAX_COLUMNS_IN_ROW = 3; private static final int MAX_COLUMNS_IN_ROW = 3;
private int mNumRows; private int mNumRows;
public Drawable mDivider;
public int mDividerWidth;
public int layout(SuggestedWords suggestions, int fromPos, int maxWidth, int minWidth, public int layout(SuggestedWords suggestions, int fromPos, int maxWidth, int minWidth,
int maxRow, KeyboardView view) { int maxRow, KeyboardView view) {
clearKeys(); clearKeys();
final Paint paint = new Paint(); final Paint paint = new Paint();
paint.setAntiAlias(true); paint.setAntiAlias(true);
final int padding = (int) view.getContext().getResources() final Resources res = view.getContext().getResources();
.getDimension(R.dimen.more_suggestions_key_horizontal_padding); mDivider = res.getDrawable(R.drawable.more_suggestions_divider);
// TODO: Drawable itself should has an alpha value.
mDivider.setAlpha(128);
mDividerWidth = mDivider.getIntrinsicWidth();
final int padding = (int) res.getDimension(
R.dimen.more_suggestions_key_horizontal_padding);
int row = 0; int row = 0;
int pos = fromPos, rowStartPos = fromPos; int pos = fromPos, rowStartPos = fromPos;
@ -66,8 +75,10 @@ public class MoreSuggestions extends Keyboard {
// TODO: Should take care of text x-scaling. // TODO: Should take care of text x-scaling.
mWidths[pos] = (int)view.getDefaultLabelWidth(word, paint) + padding; mWidths[pos] = (int)view.getDefaultLabelWidth(word, paint) + padding;
final int numColumn = pos - rowStartPos + 1; final int numColumn = pos - rowStartPos + 1;
final int columnWidth =
(maxWidth - mDividerWidth * (numColumn - 1)) / numColumn;
if (numColumn > MAX_COLUMNS_IN_ROW if (numColumn > MAX_COLUMNS_IN_ROW
|| !fitInWidth(rowStartPos, pos + 1, maxWidth / numColumn)) { || !fitInWidth(rowStartPos, pos + 1, columnWidth)) {
if ((row + 1) >= maxRow) { if ((row + 1) >= maxRow) {
break; break;
} }
@ -98,13 +109,14 @@ public class MoreSuggestions extends Keyboard {
int maxRowWidth = 0; int maxRowWidth = 0;
int pos = startPos; int pos = startPos;
for (int row = 0; row < mNumRows; row++) { for (int row = 0; row < mNumRows; row++) {
final int numColumn = mNumColumnsInRow[row]; final int numColumnInRow = mNumColumnsInRow[row];
int maxKeyWidth = 0; int maxKeyWidth = 0;
while (pos < endPos && mRowNumbers[pos] == row) { while (pos < endPos && mRowNumbers[pos] == row) {
maxKeyWidth = Math.max(maxKeyWidth, mWidths[pos]); maxKeyWidth = Math.max(maxKeyWidth, mWidths[pos]);
pos++; pos++;
} }
maxRowWidth = Math.max(maxRowWidth, maxKeyWidth * numColumn); maxRowWidth = Math.max(maxRowWidth,
maxKeyWidth * numColumnInRow + mDividerWidth * (numColumnInRow - 1));
} }
return maxRowWidth; return maxRowWidth;
} }
@ -115,15 +127,19 @@ public class MoreSuggestions extends Keyboard {
{ 2, 0, 1}, { 2, 0, 1},
}; };
private int getColumnNumber(int pos) { public int getNumColumnInRow(int pos) {
return mNumColumnsInRow[mRowNumbers[pos]];
}
public int getColumnNumber(int pos) {
final int columnOrder = mColumnOrders[pos]; final int columnOrder = mColumnOrders[pos];
final int numColumn = mNumColumnsInRow[mRowNumbers[pos]]; final int numColumn = getNumColumnInRow(pos);
return COLUMN_ORDER_TO_NUMBER[numColumn - 1][columnOrder]; return COLUMN_ORDER_TO_NUMBER[numColumn - 1][columnOrder];
} }
public int getX(int pos) { public int getX(int pos) {
final int columnNumber = getColumnNumber(pos); final int columnNumber = getColumnNumber(pos);
return columnNumber * getWidth(pos); return columnNumber * (getWidth(pos) + mDividerWidth);
} }
public int getY(int pos) { public int getY(int pos) {
@ -132,9 +148,8 @@ public class MoreSuggestions extends Keyboard {
} }
public int getWidth(int pos) { public int getWidth(int pos) {
final int row = mRowNumbers[pos]; final int numColumnInRow = getNumColumnInRow(pos);
final int numColumn = mNumColumnsInRow[row]; return (mWidth - mDividerWidth * (numColumnInRow - 1)) / numColumnInRow;
return mWidth / numColumn;
} }
public int getFlags(int pos) { public int getFlags(int pos) {
@ -146,11 +161,11 @@ public class MoreSuggestions extends Keyboard {
if (row == mNumRows - 1) if (row == mNumRows - 1)
rowFlags |= Keyboard.EDGE_TOP; rowFlags |= Keyboard.EDGE_TOP;
final int numColumn = mNumColumnsInRow[row]; final int numColumnInRow = mNumColumnsInRow[row];
final int column = getColumnNumber(pos); final int column = getColumnNumber(pos);
if (column == 0) if (column == 0)
rowFlags |= Keyboard.EDGE_LEFT; rowFlags |= Keyboard.EDGE_LEFT;
if (column == numColumn - 1) if (column == numColumnInRow - 1)
rowFlags |= Keyboard.EDGE_RIGHT; rowFlags |= Keyboard.EDGE_RIGHT;
return rowFlags; return rowFlags;
@ -190,13 +205,23 @@ public class MoreSuggestions extends Keyboard {
public MoreSuggestions build() { public MoreSuggestions build() {
final MoreSuggestionsParam params = mParams; final MoreSuggestionsParam params = mParams;
for (int pos = mFromPos; pos < mToPos; pos++) { for (int pos = mFromPos; pos < mToPos; pos++) {
final int x = params.getX(pos);
final int y = params.getY(pos);
final int width = params.getWidth(pos);
final String word = mSuggestions.getWord(pos).toString(); final String word = mSuggestions.getWord(pos).toString();
final String info = getDebugInfo(mSuggestions, pos); final String info = getDebugInfo(mSuggestions, pos);
final int index = pos + SUGGESTION_CODE_BASE; final int index = pos + SUGGESTION_CODE_BASE;
final Key key = new Key( final Key key = new Key(
params, word, info, null, index, null, params.getX(pos), params.getY(pos), params, word, info, null, index, null, x, y, width,
params.getWidth(pos), params.mDefaultRowHeight, params.getFlags(pos)); params.mDefaultRowHeight, params.getFlags(pos));
params.onAddKey(key); params.onAddKey(key);
final int columnNumber = params.getColumnNumber(pos);
final int numColumnInRow = params.getNumColumnInRow(pos);
if (columnNumber < numColumnInRow - 1) {
final Key.Spacer spacer = new Key.Spacer(params, params.mDivider, x + width, y,
params.mDividerWidth, params.mDefaultRowHeight);
params.onAddKey(spacer);
}
} }
return new MoreSuggestions(params); return new MoreSuggestions(params);
} }