From 1cbc4eec52a7c29055b2bb26c8c5278468e3a46b Mon Sep 17 00:00:00 2001 From: Mihai Branescu Date: Fri, 24 Jul 2020 19:14:04 +0300 Subject: [PATCH] For #7716 - modified xml to let 'Learn more' button have min 48dp for a11y Added a margin top of 20dp as well on the buttons so as not to change UI --- app/src/main/res/layout/search_suggestions_onboarding.xml | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/app/src/main/res/layout/search_suggestions_onboarding.xml b/app/src/main/res/layout/search_suggestions_onboarding.xml index a4cf42fd1..d957c2c13 100644 --- a/app/src/main/res/layout/search_suggestions_onboarding.xml +++ b/app/src/main/res/layout/search_suggestions_onboarding.xml @@ -50,12 +50,13 @@ android:id="@+id/learn_more" android:layout_width="0dp" android:layout_height="wrap_content" + android:minHeight="48dp" android:text="@string/exceptions_empty_message_learn_more_link" android:textAppearance="?android:attr/textAppearanceSmall" android:textColor="?attr/accentHighContrast" android:textStyle="bold" app:layout_constraintBottom_toTopOf="@id/allow" - app:layout_constraintEnd_toEndOf="@id/title" + app:layout_constraintEnd_toStartOf="@id/dismiss" app:layout_constraintStart_toStartOf="@id/title" app:layout_constraintTop_toBottomOf="@id/text" tools:textColor="@color/accent_high_contrast_private_theme"/> @@ -67,9 +68,9 @@ android:layout_height="wrap_content" android:padding="12dp" android:text="@string/search_suggestions_onboarding_allow_button" - app:layout_constraintBottom_toBottomOf="parent" + android:layout_marginTop="20dp" app:layout_constraintEnd_toEndOf="@id/title" - app:layout_constraintTop_toBottomOf="@id/learn_more" /> + app:layout_constraintTop_toBottomOf="@id/text" />