UI fixes (#85)

* Divider line color, Resizing of Location selection, Material Drawer PrimaryTextSize Signed-off-by: Jeevan Surendran <jvns67@gmail.com> * created separate dimensions Signed-off-by: Jeevan Surendran <jvns67@gmail.com>
checkin-team · Mar 19, 2020 · 1b4082d · 1b4082d
1 parent 68b20d5
commit 1b4082d
Show file tree

Hide file tree

Showing 5 changed files with 21 additions and 12 deletions.
diff --git a/app/src/main/res/layout/activity_home.xml b/app/src/main/res/layout/activity_home.xml
@@ -30,38 +30,43 @@
 
                     <ImageView
                         android:id="@+id/im_checkin_logo"
-                        android:layout_width="@dimen/width_checkin_logo_icon"
+                        android:layout_width="wrap_content"
                         android:layout_height="@dimen/height_checkin_logo_icon"
                         android:layout_gravity="center"
                         android:adjustViewBounds="true"
                         android:src="@drawable/check_in_logo_no_background"
                         app:layout_constraintBottom_toBottomOf="parent"
                         app:layout_constraintStart_toStartOf="parent"
                         app:layout_constraintTop_toTopOf="parent" />
-    
+
                     <TextView
                         android:id="@+id/tv_checkin_location_heading"
                         android:layout_width="wrap_content"
                         android:layout_height="wrap_content"
                         android:layout_marginStart="@dimen/spacing_small"
-                        android:layout_marginTop="@dimen/spacing_unit"
+                        android:layout_marginBottom="@dimen/spacing_small"
                         android:fontFamily="@font/opensans_semibold"
                         android:text="Location"
                         android:textSize="@dimen/font_extra_small"
+                        android:translationY="-4dp"
+                        app:layout_constraintBottom_toBottomOf="@+id/im_checkin_logo"
                         app:layout_constraintStart_toEndOf="@id/im_checkin_logo"
-                        app:layout_constraintTop_toTopOf="@id/im_checkin_logo" />
-
+                        app:layout_constraintTop_toTopOf="@id/im_checkin_logo"
+                        app:layout_constraintVertical_bias="0.0" />
+
                     <TextView
                         android:id="@+id/tv_checkin_location"
                         style="@style/TealBlueTextStyle"
                         android:layout_width="wrap_content"
                         android:layout_height="wrap_content"
-                        android:layout_marginBottom="@dimen/spacing_unit"
-                        android:text="Bengaluru"
                         android:fontFamily="@font/josefinsans_semibold"
-                        android:textSize="@dimen/font_large"
+                        android:text="Bengaluru"
+                        android:textSize="@dimen/font_heading"
+                        android:translationY="3dp"
                         app:layout_constraintBottom_toBottomOf="@id/im_checkin_logo"
-                        app:layout_constraintStart_toStartOf="@id/tv_checkin_location_heading" />
+                        app:layout_constraintStart_toStartOf="@id/tv_checkin_location_heading"
+                        app:layout_constraintTop_toTopOf="@id/im_checkin_logo"
+                        app:layout_constraintVertical_bias="1.0" />
                 </androidx.constraintlayout.widget.ConstraintLayout>
             </androidx.appcompat.widget.Toolbar>
 

diff --git a/app/src/main/res/layout/fragment_user_location_switch.xml b/app/src/main/res/layout/fragment_user_location_switch.xml
@@ -59,6 +59,8 @@
                 android:layout_marginEnd="@dimen/spacing_small"
                 android:background="@null"
                 android:padding="@dimen/spacing_extra_small"
+                android:inputType="text"
+                android:maxLines="1"
                 android:textColor="@color/pinkish_grey"
                 app:layout_constraintBottom_toBottomOf="parent"
                 app:layout_constraintEnd_toStartOf="@id/im_user_location_cross"

diff --git a/app/src/main/res/layout/item_transaction_details.xml b/app/src/main/res/layout/item_transaction_details.xml
@@ -91,7 +91,7 @@
             android:layout_width="@dimen/spacing_unit"
             android:layout_height="0dp"
             android:layout_marginEnd="125dp"
-            android:background="@color/pinkish_grey"
+            android:background="@color/lightest_grey"
             app:layout_constraintBottom_toBottomOf="parent"
             app:layout_constraintEnd_toEndOf="parent"
             app:layout_constraintTop_toTopOf="parent" />

diff --git a/app/src/main/res/values-sw400dp/dimens.xml b/app/src/main/res/values-sw400dp/dimens.xml
@@ -189,7 +189,7 @@
 
     <!--Home -->
     <dimen name="width_checkin_logo_icon">28dp</dimen>
-    <dimen name="height_checkin_logo_icon">36dp</dimen>
+    <dimen name="height_checkin_logo_icon">26dp</dimen>
     <dimen name="width_trending_rest">150dp</dimen>
     <dimen name="height_trending_rest">140dp</dimen>
     <dimen name="size_qr_scanner">20dp</dimen>

diff --git a/app/src/main/res/values/dimens.xml b/app/src/main/res/values/dimens.xml
@@ -216,7 +216,7 @@
 
     <!--Home -->
     <dimen name="width_checkin_logo_icon">26dp</dimen>
-    <dimen name="height_checkin_logo_icon">34dp</dimen>
+    <dimen name="height_checkin_logo_icon">24dp</dimen>
     <dimen name="width_trending_rest">120dp</dimen>
     <dimen name="height_trending_rest">110dp</dimen>
     <dimen name="size_qr_scanner">20dp</dimen>
@@ -234,6 +234,8 @@
 
     <dimen name="new_bottom_user_height">800dp</dimen>
 
+    <!-- Material Drawer -->
+    <dimen name="material_drawer_item_primary_text">16sp</dimen>
     <!--Location-->
     <dimen name="width_current_location">150dp</dimen>
     <dimen name="height_current_location">140dp</dimen>