Commit 9196ef69 authored by Tommy Martino's avatar Tommy Martino Committed by Commit Bot

[AF Android] Fix height of multiline rows

Multiline entries are used mostly for warnings and other edge cases
where descriptive text is needed. By their nature the height is
variable, so we need to set padding and use a content-wrapping height.
This was done in the old implementation but errantly not carried over to
the refactor.

Bug: 903554
Change-Id: I6e9873c757fb9087af3066bb55017badccd15149
Reviewed-on: https://chromium-review.googlesource.com/c/1327579Reviewed-by: default avatarTheresa <twellington@chromium.org>
Commit-Queue: Tommy Martino <tmartino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#607724}
parent 6e67f5d0
...@@ -16,11 +16,14 @@ ...@@ -16,11 +16,14 @@
android:orientation="horizontal" android:orientation="horizontal"
tools:ignore="UnusedResources" > tools:ignore="UnusedResources" >
<!-- TODO(crbug.com/903554): Remove the fixed layout_height in favor of an equivalent line
height and padding. -->
<LinearLayout <LinearLayout
android:id="@+id/dropdown_label_wrapper" android:id="@+id/dropdown_label_wrapper"
android:layout_width="0dp" android:layout_width="0dp"
android:layout_height="@dimen/autofill_dropdown_refresh_item_height" android:layout_height="wrap_content"
android:layout_weight="1" android:layout_weight="1"
android:minHeight="@dimen/autofill_dropdown_refresh_item_height"
android:gravity="center_vertical" android:gravity="center_vertical"
android:orientation="vertical" > android:orientation="vertical" >
......
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