From d05145553b487e00ed30377edaaf4b5eb7af4eb5 Mon Sep 17 00:00:00 2001
From: shaoweishen <shaoweishen@google.com>
Date: Thu, 9 Jan 2025 08:40:24 +0000
Subject: [PATCH] [Physical Keyboard] Fix bugs

adjust properties of dialog to match requirements

Bug: 374035349
Bug: 374030364
Test: atest SettingsRoboTests
Flag: com.android.settings.keyboard.keyboard_and_touchpad_a11y_new_page_enabled
Change-Id: I4ee77a7ea249e2bd00ef5a05270c1ab3f257f547
---
 ...dialog_keyboard_a11y_input_setting_keys.xml | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/res/layout/dialog_keyboard_a11y_input_setting_keys.xml b/res/layout/dialog_keyboard_a11y_input_setting_keys.xml
index dfa3c46603d..d826fee96fd 100644
--- a/res/layout/dialog_keyboard_a11y_input_setting_keys.xml
+++ b/res/layout/dialog_keyboard_a11y_input_setting_keys.xml
@@ -55,7 +55,7 @@
             android:text="@string/input_setting_keys_dialog_option_200"
             android:paddingStart="12dp"
             android:layout_width="match_parent"
-            android:layout_height="wrap_content"
+            android:layout_height="48dp"
             android:layout_gravity="start|center_vertical"
             android:background="@null"/>
         <RadioButton
@@ -63,30 +63,30 @@
             android:text="@string/input_setting_keys_dialog_option_400"
             android:paddingStart="12dp"
             android:layout_width="match_parent"
-            android:layout_height="wrap_content"
+            android:layout_height="48dp"
             android:layout_gravity="start|center_vertical"
-            android:layout_marginTop="12dp"
+            android:layout_marginTop="6dp"
             android:background="@null"/>
         <RadioButton
             android:id="@+id/input_setting_keys_value_600"
             android:text="@string/input_setting_keys_dialog_option_600"
             android:paddingStart="12dp"
             android:layout_width="match_parent"
-            android:layout_height="wrap_content"
+            android:layout_height="48dp"
             android:layout_gravity="start|center_vertical"
-            android:layout_marginTop="12dp"
+            android:layout_marginTop="6dp"
             android:background="@null"/>
 
         <LinearLayout
             android:orientation="horizontal"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:layout_marginVertical="12dp">
+            android:layout_marginVertical="6dp">
             <RadioButton
                 android:id="@+id/input_setting_keys_value_custom"
                 android:paddingStart="12dp"
                 android:layout_width="wrap_content"
-                android:layout_height="wrap_content"
+                android:layout_height="48dp"
                 android:layout_gravity="center_vertical"
                 android:background="@null"/>
             <LinearLayout
@@ -94,6 +94,7 @@
                 android:orientation="vertical"
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content"
+                android:minHeight="48dp"
                 android:layout_gravity="center_vertical"
                 android:gravity="center_vertical"
                 android:paddingEnd="16dp">
@@ -123,8 +124,7 @@
                     android:layout_height="wrap_content"
                     android:layout_gravity="center_vertical"
                     android:layout_marginTop="8dp"
-                    android:visibility="gone"
-                    android:background="@null"/>
+                    android:visibility="gone"/>
             </LinearLayout>
         </LinearLayout>
     </RadioGroup>