EVISACCESSAPP-6111: Small UI improvements

nevissecurity · Aug 6, 2024 · d415af5 · d415af5
1 parent bf12075
commit d415af5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 12 deletions.
diff --git a/app/src/main/res/layout/item_account.xml b/app/src/main/res/layout/item_account.xml
@@ -16,16 +16,14 @@
     <androidx.constraintlayout.widget.ConstraintLayout
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
+        android:background="?android:selectableItemBackground"
+        android:padding="16dp"
         android:onClick="@{() -> accountSelectedListener.onAccountSelected(account)}">
 
         <TextView
             android:id="@+id/usernameTextView"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
-            android:layout_marginStart="16dp"
-            android:layout_marginTop="16dp"
-            android:layout_marginEnd="16dp"
-            android:layout_marginBottom="16dp"
             android:text="@{account.username()}"
             android:textSize="16sp"
             android:textStyle="bold"
@@ -34,4 +32,4 @@
             app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toTopOf="parent" />
     </androidx.constraintlayout.widget.ConstraintLayout>
-</layout>
+</layout>
diff --git a/app/src/main/res/layout/item_authenticator.xml b/app/src/main/res/layout/item_authenticator.xml
@@ -29,15 +29,14 @@
     <androidx.constraintlayout.widget.ConstraintLayout
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
+        android:background="?android:selectableItemBackground"
+        android:padding="16dp"
         android:onClick="@{() -> authenticatorSelectedListener.onAuthenticatorSelected(aaid)}">
 
         <TextView
             android:id="@+id/titleTextView"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
-            android:layout_marginStart="16dp"
-            android:layout_marginTop="16dp"
-            android:layout_marginEnd="16dp"
             android:enabled="@{isEnabled}"
             android:text="@{titleResId}"
             android:textSize="16sp"
@@ -50,10 +49,7 @@
             android:id="@+id/messageTextView"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
-            android:layout_marginStart="16dp"
             android:layout_marginTop="8dp"
-            android:layout_marginEnd="16dp"
-            android:layout_marginBottom="16dp"
             android:enabled="@{isEnabled}"
             android:text='@{message ?? ""}'
             android:textSize="14sp"
@@ -64,4 +60,4 @@
             app:layout_constraintTop_toBottomOf="@id/titleTextView" />
 
     </androidx.constraintlayout.widget.ConstraintLayout>
-</layout>
+</layout>