From 89c58745d639a4434dfbe600c28dad1bc8bc6888 Mon Sep 17 00:00:00 2001
From: "Tadashi G. Takaoka" <takaoka@google.com>
Date: Wed, 8 Dec 2010 20:05:34 +0900
Subject: [PATCH] Tweak candidate view height and etc.

Change-Id: I91963d0be122baf2274ea03b5028a66259a2cdb6
---
 java/res/values-xlarge-land/dimens.xml        |  3 ++-
 java/res/values-xlarge/dimens.xml             |  8 +++----
 .../inputmethod/latin/CandidateView.java      | 23 ++++++++++---------
 3 files changed, 18 insertions(+), 16 deletions(-)

diff --git a/java/res/values-xlarge-land/dimens.xml b/java/res/values-xlarge-land/dimens.xml
index e1a53f9dce..5b9d50abc0 100644
--- a/java/res/values-xlarge-land/dimens.xml
+++ b/java/res/values-xlarge-land/dimens.xml
@@ -19,7 +19,8 @@
 -->
 
 <resources>
-    <dimen name="key_label_text_size">18dip</dimen>
+    <dimen name="key_letter_size">28dip</dimen>
+    <dimen name="key_label_text_size">20dip</dimen>
     <!-- left or right padding of label alignment -->
     <dimen name="key_label_horizontal_alignment_padding">18dip</dimen>
     <dimen name="candidate_strip_padding">40.0mm</dimen>
diff --git a/java/res/values-xlarge/dimens.xml b/java/res/values-xlarge/dimens.xml
index 58d7d32370..dbdfa9631f 100644
--- a/java/res/values-xlarge/dimens.xml
+++ b/java/res/values-xlarge/dimens.xml
@@ -36,13 +36,13 @@
     <!-- popup_key_height x -1.0 -->
     <dimen name="mini_keyboard_vertical_correction">-13.0mm</dimen>
 
-    <dimen name="key_letter_size">24dip</dimen>
-    <dimen name="key_label_text_size">14dip</dimen>
+    <dimen name="key_letter_size">26dip</dimen>
+    <dimen name="key_label_text_size">16dip</dimen>
     <dimen name="key_preview_text_size_large">24dip</dimen>
     <!-- left or right padding of label alignment -->
-    <dimen name="key_label_horizontal_alignment_padding">4dip</dimen>
+    <dimen name="key_label_horizontal_alignment_padding">6dip</dimen>
 
-    <dimen name="candidate_strip_height">56dip</dimen>
+    <dimen name="candidate_strip_height">46dip</dimen>
     <dimen name="candidate_strip_padding">15.0mm</dimen>
     <dimen name="candidate_text_size">22dip</dimen>
 </resources>
diff --git a/java/src/com/android/inputmethod/latin/CandidateView.java b/java/src/com/android/inputmethod/latin/CandidateView.java
index 460ef8650d..3196fa65d5 100644
--- a/java/src/com/android/inputmethod/latin/CandidateView.java
+++ b/java/src/com/android/inputmethod/latin/CandidateView.java
@@ -138,19 +138,20 @@ public class CandidateView extends LinearLayout implements OnClickListener, OnLo
             TextView tv = (TextView)v.findViewById(R.id.candidate_word);
             tv.setTypeface(Typeface.DEFAULT);
             tv.setTextColor(mColorNormal);
-            if (mConfigCandidateHighlightFontColorEnabled) {
-                if (haveMinimalSuggestion
-                        && ((i == 1 && !typedWordValid) || (i == 0 && typedWordValid))) {
-                    tv.setTypeface(Typeface.DEFAULT_BOLD);
+            if (haveMinimalSuggestion
+                    && ((i == 1 && !typedWordValid) || (i == 0 && typedWordValid))) {
+                // TODO: Display underline for the auto-correction word
+                tv.setTypeface(Typeface.DEFAULT_BOLD);
+                if (mConfigCandidateHighlightFontColorEnabled)
                     tv.setTextColor(mColorRecommended);
-                    existsAutoCompletion = true;
-                } else if (i != 0 || (wordLength == 1 && count > 1)) {
-                    // HACK: even if i == 0, we use mColorOther when this suggestion's length is 1
-                    // and there are multiple suggestions, such as the default punctuation list.
+                existsAutoCompletion = true;
+            } else if (i != 0 || (wordLength == 1 && count > 1)) {
+                // HACK: even if i == 0, we use mColorOther when this
+                // suggestion's length is 1
+                // and there are multiple suggestions, such as the default
+                // punctuation list.
+                if (mConfigCandidateHighlightFontColorEnabled)
                     tv.setTextColor(mColorOther);
-                }
-            } else {
-                // TODO: Display underline for the auto-correction word
             }
             tv.setText(suggestion);
             tv.setClickable(true);
-- 
GitLab