Commit 3c77b67a authored by Friedrich Horschig's avatar Friedrich Horschig Committed by Commit Bot

[Mfill Android] Fix accessory fallback sheet paddings

This CL removes hard-coded heights and paddings. They are superfluous
since ChipView enforces its padding and restricts the line height.

This means the duplicate paddings (see screenshots in bug) are gone.

Bug: 997611
Change-Id: I805792254fced2c25c63fa75b4bc48021ff90fa2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769440Reviewed-by: default avatarJan Wilken Dörrie <jdoerrie@chromium.org>
Commit-Queue: Friedrich [CET] <fhorschig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690328}
parent 4306aab9
......@@ -46,7 +46,7 @@
<LinearLayout
android:gravity="center_vertical|start"
android:fillViewport="true"
android:layout_height="@dimen/keyboard_accessory_suggestion_height"
android:layout_height="wrap_content"
android:layout_width="match_parent"
android:orientation="horizontal">
......@@ -78,8 +78,6 @@
<org.chromium.ui.widget.ChipView
android:id="@+id/address_home_country"
android:gravity="center_vertical|start"
android:layout_marginTop="@dimen/keyboard_accessory_sheet_padding"
android:layout_marginBottom="@dimen/keyboard_accessory_sheet_padding"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
style="@style/InputChip" />
......
......@@ -34,7 +34,6 @@
<org.chromium.ui.widget.ChipView
android:id="@+id/cc_number"
android:gravity="center_vertical|start"
android:layout_marginBottom="@dimen/keyboard_accessory_sheet_padding"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
style="@style/InputChip" />
......@@ -43,7 +42,7 @@
android:id="@+id/exp_group"
android:gravity="center_vertical|start"
android:fillViewport="true"
android:layout_height="@dimen/keyboard_accessory_suggestion_height"
android:layout_height="wrap_content"
android:layout_width="match_parent"
android:orientation="horizontal">
......@@ -77,7 +76,6 @@
<org.chromium.ui.widget.ChipView
android:id="@+id/cardholder"
android:gravity="center_vertical|start"
android:layout_marginTop="@dimen/keyboard_accessory_sheet_padding"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
style="@style/InputChip" />
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment