Fix suggestions strip press state to align vertically

This change also tweaks more_suggestions_hint drawables to eliminate
unnecessarily padding at the bottom.

Bug: 5204563
Change-Id: If5ec731c9a490da0116a020e8fd405c7d025d707

4 files changed, 5 insertions, 4 deletions

diff --git a/java/res/drawable-hdpi/more_suggestions_hint.png b/java/res/drawable-hdpi/more_suggestions_hint.png
index 4515f4434..93604752b 100644
--- a/java/res/drawable-hdpi/more_suggestions_hint.png
+++ b/java/res/drawable-hdpi/more_suggestions_hint.png
diff --git a/java/res/drawable-mdpi/more_suggestions_hint.png b/java/res/drawable-mdpi/more_suggestions_hint.png
index 6168de353..7352810d3 100644
--- a/java/res/drawable-mdpi/more_suggestions_hint.png
+++ b/java/res/drawable-mdpi/more_suggestions_hint.png
diff --git a/java/res/drawable-xhdpi/more_suggestions_hint.png b/java/res/drawable-xhdpi/more_suggestions_hint.png
index f577a3617..35fb42087 100644
--- a/java/res/drawable-xhdpi/more_suggestions_hint.png
+++ b/java/res/drawable-xhdpi/more_suggestions_hint.png
diff --git a/java/src/com/android/inputmethod/latin/CandidateView.java b/java/src/com/android/inputmethod/latin/CandidateView.java
index d46b4b5b5..5f20c70b4 100644
--- a/java/src/com/android/inputmethod/latin/CandidateView.java
+++ b/java/src/com/android/inputmethod/latin/CandidateView.java
@@ -413,8 +413,10 @@ public class CandidateView extends LinearLayout implements OnClickListener, OnLo
                     // TODO: This "more suggestions hint" should have nicely designed icon.
                     word.setCompoundDrawablesWithIntrinsicBounds(
                             null, null, null, mMoreCandidateHint);
+                    // HACK: To align with other TextView that has no compound drawables.
+                    word.setCompoundDrawablePadding(-mMoreCandidateHint.getIntrinsicHeight());
                 } else {
-                    word.setCompoundDrawables(null, null, null, null);
+                    word.setCompoundDrawablesWithIntrinsicBounds(null, null, null, null);
                 }
 
                 // Disable this candidate if the suggestion is null or empty.
@@ -426,7 +428,7 @@ public class CandidateView extends LinearLayout implements OnClickListener, OnLo
                 word.setText(text); // TextView.setText() resets text scale x to 1.0.
                 word.setTextScaleX(scaleX);
                 stripView.addView(word);
-                setLayoutWeight(word, getCandidateWeight(index), mCandidateStripHeight);
+                setLayoutWeight(word, getCandidateWeight(index), MATCH_PARENT);
 
                 if (DBG) {
                     final CharSequence debugInfo = getDebugInfo(suggestions, pos);
@@ -526,8 +528,7 @@ public class CandidateView extends LinearLayout implements OnClickListener, OnLo
 
         mPreviewPopup = new PopupWindow(context);
         mPreviewText = (TextView) inflater.inflate(R.layout.candidate_preview, null);
-        mPreviewPopup.setWindowLayoutMode(ViewGroup.LayoutParams.WRAP_CONTENT,
-                ViewGroup.LayoutParams.WRAP_CONTENT);
+        mPreviewPopup.setWindowLayoutMode(WRAP_CONTENT, WRAP_CONTENT);
         mPreviewPopup.setContentView(mPreviewText);
         mPreviewPopup.setBackgroundDrawable(null);