Take optional fields into consideration when measuring

Also, move the padding to the icon so it's not dependent on an optional
view.

Bug: 123378871
Test: Do not set subtitle/description. Dialog layout looks normal now.
Change-Id: Ia6ae768660442aeaadeb09aa7117780b8a97d56f
This commit is contained in:
Kevin Chyn
2019-09-18 11:47:24 -07:00
parent d6aae0f528
commit c2ea8f264f
2 changed files with 5 additions and 2 deletions

View File

@@ -42,7 +42,6 @@
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:paddingHorizontal="24dp"
android:paddingBottom="48dp"
android:paddingTop="8dp"
android:gravity="@integer/biometric_dialog_text_gravity"
android:textSize="16sp"
@@ -52,6 +51,7 @@
android:id="@+id/biometric_icon"
android:layout_width="@dimen/biometric_dialog_biometric_icon_size"
android:layout_height="@dimen/biometric_dialog_biometric_icon_size"
android:paddingTop="48dp"
android:layout_gravity="center_horizontal"
android:scaleType="fitXY" />

View File

@@ -608,7 +608,10 @@ public abstract class AuthBiometricView extends LinearLayout {
MeasureSpec.makeMeasureSpec(newWidth, MeasureSpec.EXACTLY),
MeasureSpec.makeMeasureSpec(height, MeasureSpec.AT_MOST));
}
totalHeight += child.getMeasuredHeight();
if (child.getVisibility() != View.GONE) {
totalHeight += child.getMeasuredHeight();
}
}
// Use the new width so it's centered horizontally