Merge "Revert "Prepare for changing text color for functional keys""

main
Tadashi G. Takaoka 2014-02-24 08:54:54 +00:00 committed by Android (Google) Code Review
commit b08a9e021c
5 changed files with 8 additions and 77 deletions

View File

@ -1,48 +0,0 @@
<?xml version="1.0" encoding="utf-8"?>
<!-- Copyright (C) 2013 The Android Open Source Project
Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
-->
<selector xmlns:android="http://schemas.android.com/apk/res/android">
<!-- Functional keys. -->
<item android:state_single="true" android:state_pressed="true"
android:color="@color/key_text_color_functional_holo" />
<item android:state_single="true"
android:color="@color/key_text_color_functional_holo" />
<!-- Action keys. -->
<item android:state_active="true" android:state_pressed="true"
android:color="@color/key_text_color_normal_holo" />
<item android:state_active="true"
android:color="@color/key_text_color_normal_holo" />
<!-- Toggle keys. Use checkable/checked state. -->
<item android:state_checkable="true" android:state_checked="true" android:state_pressed="true"
android:color="@color/key_text_color_normal_holo" />
<item android:state_checkable="true" android:state_pressed="true"
android:color="@color/key_text_color_normal_holo" />
<item android:state_checkable="true" android:state_checked="true"
android:color="@color/key_text_color_normal_holo" />
<item android:state_checkable="true"
android:color="@color/key_text_color_normal_holo" />
<!-- Empty background keys. -->
<item android:state_empty="true"
android:color="@color/key_text_color_normal_holo" />
<!-- Normal keys. -->
<item android:state_pressed="true"
android:color="@color/key_text_color_normal_holo" />
<item android:color="@color/key_text_color_normal_holo" />
</selector>

View File

@ -39,6 +39,7 @@
<color name="typed_word_color_ics">#D833B5E5</color> <color name="typed_word_color_ics">#D833B5E5</color>
<color name="suggested_word_color_ics">#B233B5E5</color> <color name="suggested_word_color_ics">#B233B5E5</color>
<color name="highlight_translucent_color_ics">#9933B5E5</color> <color name="highlight_translucent_color_ics">#9933B5E5</color>
<color name="key_text_color_holo">@android:color/white</color>
<color name="key_text_shadow_color_holo">@android:color/transparent</color> <color name="key_text_shadow_color_holo">@android:color/transparent</color>
<color name="key_text_inactivated_color_holo">#66E0E4E5</color> <color name="key_text_inactivated_color_holo">#66E0E4E5</color>
<color name="key_hint_letter_color_holo">#80000000</color> <color name="key_hint_letter_color_holo">#80000000</color>
@ -65,7 +66,4 @@
<!-- TODO: Color which should be included in the theme --> <!-- TODO: Color which should be included in the theme -->
<color name="emoji_key_background_color">#00000000</color> <color name="emoji_key_background_color">#00000000</color>
<color name="emoji_key_pressed_background_color">#30FFFFFF</color> <color name="emoji_key_pressed_background_color">#30FFFFFF</color>
<color name="key_text_color_normal_holo">@android:color/white</color>
<color name="key_text_color_functional_holo">@android:color/white</color>
</resources> </resources>

View File

@ -137,8 +137,6 @@ public class Key implements Comparable<Key> {
private final OptionalAttributes mOptionalAttributes; private final OptionalAttributes mOptionalAttributes;
private static final int DEFAULT_TEXT_COLOR = 0xFFFFFFFF;
private static final class OptionalAttributes { private static final class OptionalAttributes {
/** Text to output when pressed. This can be multiple characters, like ".com" */ /** Text to output when pressed. This can be multiple characters, like ".com" */
public final String mOutputText; public final String mOutputText;
@ -585,22 +583,7 @@ public class Key implements Comparable<Key> {
} }
public final int selectTextColor(final KeyDrawParams params) { public final int selectTextColor(final KeyDrawParams params) {
if (isShiftedLetterActivated()) { return isShiftedLetterActivated() ? params.mTextInactivatedColor : params.mTextColor;
return params.mTextInactivatedColor;
}
if (params.mTextColorStateList == null) {
return DEFAULT_TEXT_COLOR;
}
final int[] state;
// TODO: Hack!!!!!!!! Consider having a new attribute for the functional text labels.
// Currently, we distinguish "input key" from "functional key" by checking the
// length of the label( > 1) and "functional" attributes (= true).
if (mLabel != null && mLabel.length() > 1) {
state = getCurrentDrawableState();
} else {
state = KEY_STATE_NORMAL;
}
return params.mTextColorStateList.getColorForState(state, DEFAULT_TEXT_COLOR);
} }
public final int selectHintTextSize(final KeyDrawParams params) { public final int selectHintTextSize(final KeyDrawParams params) {

View File

@ -16,7 +16,6 @@
package com.android.inputmethod.keyboard.internal; package com.android.inputmethod.keyboard.internal;
import android.content.res.ColorStateList;
import android.graphics.Typeface; import android.graphics.Typeface;
import com.android.inputmethod.latin.utils.ResourceUtils; import com.android.inputmethod.latin.utils.ResourceUtils;
@ -33,7 +32,7 @@ public final class KeyDrawParams {
public int mHintLabelSize; public int mHintLabelSize;
public int mPreviewTextSize; public int mPreviewTextSize;
public ColorStateList mTextColorStateList; public int mTextColor;
public int mTextInactivatedColor; public int mTextInactivatedColor;
public int mTextShadowColor; public int mTextShadowColor;
public int mHintLetterColor; public int mHintLetterColor;
@ -58,7 +57,7 @@ public final class KeyDrawParams {
mHintLabelSize = copyFrom.mHintLabelSize; mHintLabelSize = copyFrom.mHintLabelSize;
mPreviewTextSize = copyFrom.mPreviewTextSize; mPreviewTextSize = copyFrom.mPreviewTextSize;
mTextColorStateList = copyFrom.mTextColorStateList; mTextColor = copyFrom.mTextColor;
mTextInactivatedColor = copyFrom.mTextInactivatedColor; mTextInactivatedColor = copyFrom.mTextInactivatedColor;
mTextShadowColor = copyFrom.mTextShadowColor; mTextShadowColor = copyFrom.mTextShadowColor;
mHintLetterColor = copyFrom.mHintLetterColor; mHintLetterColor = copyFrom.mHintLetterColor;
@ -90,8 +89,8 @@ public final class KeyDrawParams {
attr.mShiftedLetterHintRatio, mShiftedLetterHintSize); attr.mShiftedLetterHintRatio, mShiftedLetterHintSize);
mHintLabelSize = selectTextSize(keyHeight, attr.mHintLabelRatio, mHintLabelSize); mHintLabelSize = selectTextSize(keyHeight, attr.mHintLabelRatio, mHintLabelSize);
mPreviewTextSize = selectTextSize(keyHeight, attr.mPreviewTextRatio, mPreviewTextSize); mPreviewTextSize = selectTextSize(keyHeight, attr.mPreviewTextRatio, mPreviewTextSize);
mTextColorStateList =
attr.mTextColorStateList != null ? attr.mTextColorStateList : mTextColorStateList; mTextColor = selectColor(attr.mTextColor, mTextColor);
mTextInactivatedColor = selectColor(attr.mTextInactivatedColor, mTextInactivatedColor); mTextInactivatedColor = selectColor(attr.mTextInactivatedColor, mTextInactivatedColor);
mTextShadowColor = selectColor(attr.mTextShadowColor, mTextShadowColor); mTextShadowColor = selectColor(attr.mTextShadowColor, mTextShadowColor);
mHintLetterColor = selectColor(attr.mHintLetterColor, mHintLetterColor); mHintLetterColor = selectColor(attr.mHintLetterColor, mHintLetterColor);

View File

@ -16,7 +16,6 @@
package com.android.inputmethod.keyboard.internal; package com.android.inputmethod.keyboard.internal;
import android.content.res.ColorStateList;
import android.content.res.TypedArray; import android.content.res.TypedArray;
import android.graphics.Typeface; import android.graphics.Typeface;
import android.util.SparseIntArray; import android.util.SparseIntArray;
@ -38,7 +37,7 @@ public final class KeyVisualAttributes {
public final float mHintLabelRatio; public final float mHintLabelRatio;
public final float mPreviewTextRatio; public final float mPreviewTextRatio;
public final ColorStateList mTextColorStateList; public final int mTextColor;
public final int mTextInactivatedColor; public final int mTextInactivatedColor;
public final int mTextShadowColor; public final int mTextShadowColor;
public final int mHintLetterColor; public final int mHintLetterColor;
@ -119,7 +118,7 @@ public final class KeyVisualAttributes {
mPreviewTextRatio = ResourceUtils.getFraction(keyAttr, mPreviewTextRatio = ResourceUtils.getFraction(keyAttr,
R.styleable.Keyboard_Key_keyPreviewTextRatio); R.styleable.Keyboard_Key_keyPreviewTextRatio);
mTextColorStateList = keyAttr.getColorStateList(R.styleable.Keyboard_Key_keyTextColor); mTextColor = keyAttr.getColor(R.styleable.Keyboard_Key_keyTextColor, 0);
mTextInactivatedColor = keyAttr.getColor( mTextInactivatedColor = keyAttr.getColor(
R.styleable.Keyboard_Key_keyTextInactivatedColor, 0); R.styleable.Keyboard_Key_keyTextInactivatedColor, 0);
mTextShadowColor = keyAttr.getColor(R.styleable.Keyboard_Key_keyTextShadowColor, 0); mTextShadowColor = keyAttr.getColor(R.styleable.Keyboard_Key_keyTextShadowColor, 0);