Merge pull request #106 from winstonsung/dev-106

Fix various color and margin/padding issues

Author: theimpulson

app/src/main/java/app/opass/ccip/ui/LanguagePreferenceFragment.kt

@@ -15,6 +15,7 @@ import androidx.fragment.app.FragmentManager
 import androidx.recyclerview.widget.LinearLayoutManager
 import androidx.recyclerview.widget.RecyclerView
 import app.opass.ccip.R
+import app.opass.ccip.extension.updateMargin
 import com.google.android.material.dialog.MaterialAlertDialogBuilder
 
 class LanguagePreferenceFragment : DialogFragment() {
@@ -140,6 +141,14 @@ class LanguagePreferenceAdapter(
         holder.localName.text = item.localName
         holder.translatedName.text = item.translatedName
 
+        if (position == 0) {
+            holder.optionItem.updateMargin(top = 36)
+        }
+
+        if (position == items.size - 1) {
+            holder.optionItem.updateMargin(bottom = 36)
+        }
+
         if (!isSelected) {
             holder.selectedIcon.setImageDrawable(null)
         } else {

app/src/main/java/app/opass/ccip/ui/wifi/WiFiNetworkAdapter.kt

@@ -3,10 +3,13 @@ package app.opass.ccip.ui.wifi
 import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup
+import android.widget.LinearLayout
 import android.widget.TextView
 import androidx.core.view.isGone
+import androidx.core.view.marginTop
 import androidx.recyclerview.widget.RecyclerView
 import app.opass.ccip.R
+import app.opass.ccip.extension.updateMargin
 import app.opass.ccip.model.WifiNetworkInfo
 
 class WifiNetworkAdapter(
@@ -38,10 +41,19 @@ class WifiNetworkAdapter(
 
         holder.password.text = item.password
         holder.password.isGone = false
+
+        if (position == 0) {
+            holder.networkItem.updateMargin(top = 36)
+        }
+
+        if (position == items.size - 1) {
+            holder.networkItem.updateMargin(bottom = 36)
+        }
     }
 }
 
 class WifiNetworkViewHolder(view: View) : RecyclerView.ViewHolder(view) {
+    val networkItem: LinearLayout = view.findViewById(R.id.network_item)
     val name: TextView = view.findViewById(R.id.network_name)
     val password: TextView = view.findViewById(R.id.network_password)
 }

app/src/main/res/drawable/room_background.xml

@@ -2,7 +2,7 @@
 <selector xmlns:android="http://schemas.android.com/apk/res/android">
     <item>
         <shape android:shape="oval">
-            <solid android:color="@color/colorPrimary" />
+            <solid android:color="@color/backgroundColorAccent" />
         </shape>
     </item>
 </selector>

app/src/main/res/drawable/tag_round_corner.xml

@@ -2,7 +2,7 @@
 <shape xmlns:android="http://schemas.android.com/apk/res/android"
     android:shape="rectangle">
 
-    <solid android:color="?attr/colorAccent" />
+    <solid android:color="@color/colorAccentDark" />
 
     <corners android:radius="16dp" />
 </shape>

app/src/main/res/layout/activity_session_detail.xml

@@ -65,6 +65,7 @@
         android:layout_height="wrap_content"
         android:layout_margin="@dimen/fab_margin"
         android:src="@drawable/ic_star_border_white_48dp"
+        app:backgroundTint="@color/colorPrimary"
         app:layout_anchor="@id/app_bar"
         app:layout_anchorGravity="bottom|end" />
 

app/src/main/res/layout/fragment_schedule_tab.xml

@@ -26,7 +26,8 @@
         android:layout_gravity="bottom|end"
         android:layout_margin="16dp"
         android:contentDescription="@string/filter"
-        android:tint="?attr/colorOnSecondary"
+        app:backgroundTint="@color/colorPrimary"
+        android:tint="?attr/colorOnPrimary"
         android:visibility="invisible"
         app:srcCompat="@drawable/ic_baseline_filter_list_24" />
 

app/src/main/res/layout/include_auth_header.xml

@@ -18,7 +18,7 @@
 
     <Button
         android:id="@+id/not_this_event"
-        style="@style/Widget.MaterialComponents.Button.TextButton"
+        style="@style/TextButton"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:layout_gravity="center_horizontal"

app/src/main/res/layout/item_event.xml

@@ -23,15 +23,15 @@
             <RelativeLayout
                 android:layout_width="match_parent"
                 android:layout_height="150dp"
-                android:background="?attr/colorSecondary">
+                android:background="@color/backgroundColorAccent">
 
                 <ImageView
                     android:id="@+id/event_logo"
                     android:layout_width="wrap_content"
                     android:layout_height="wrap_content"
                     android:layout_centerInParent="true"
                     android:padding="10dp"
-                    card_view:srcCompat="?attr/colorSecondary" />
+                    card_view:srcCompat="@color/backgroundColorAccent" />
             </RelativeLayout>
 
             <TextView

app/src/main/res/layout/item_option_language.xml

@@ -9,17 +9,15 @@
     android:background="?attr/selectableItemBackground"
     android:focusable="true"
     android:orientation="horizontal"
-    android:paddingStart="32dp"
-    android:paddingTop="16dp"
-    android:paddingEnd="16dp"
-    android:paddingBottom="16dp">
+    android:paddingTop="8dp"
+    android:paddingBottom="8dp"
+    android:paddingStart="24dp"
+    android:paddingEnd="24dp">
 
     <LinearLayout
         android:layout_weight="1"
         android:layout_width="0dp"
         android:layout_height="wrap_content"
-        android:background="?attr/selectableItemBackground"
-        android:focusable="true"
         android:orientation="vertical">
         <TextView
             android:id="@+id/option_local_name"

app/src/main/res/layout/item_session.xml

@@ -29,9 +29,10 @@
                 android:layout_marginStart="12dp"
                 android:layout_marginTop="8dp"
                 android:autoSizeTextType="uniform"
+                android:autoSizeMinTextSize="8dp"
                 android:maxLines="1"
                 android:padding="2dp"
-                android:text="New Text"
+                android:text="TR412-1"
                 app:layout_constraintBottom_toTopOf="@id/tagsBarrier"
                 app:layout_constraintStart_toStartOf="parent"
                 app:layout_constraintTop_toTopOf="parent" />