Updates the devices list

- The unit items now react to user interaction, they display and change their appearance on click
- The adapter has a callback to manage the different options
- Minor changes to dimen values

3 files changed, 42 insertions, 10 deletions

diff --git a/androidApp/src/main/java/mx/trackermap/TrackerMap/android/devices/DevicesAdapter.kt b/androidApp/src/main/java/mx/trackermap/TrackerMap/android/devices/DevicesAdapter.kt
index 72d61e3..4aa210d 100644
--- a/androidApp/src/main/java/mx/trackermap/TrackerMap/android/devices/DevicesAdapter.kt
+++ b/androidApp/src/main/java/mx/trackermap/TrackerMap/android/devices/DevicesAdapter.kt
@@ -4,11 +4,20 @@ import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup
 import androidx.recyclerview.widget.RecyclerView
+import mx.trackermap.TrackerMap.android.R
 import mx.trackermap.TrackerMap.android.databinding.UnitItemBinding
 import mx.trackermap.TrackerMap.client.models.UnitInformation
 
-class DevicesAdapter(private val units: List<UnitInformation>) :
-    RecyclerView.Adapter<DevicesAdapter.ViewHolder>() {
+enum class Action {
+    DETAILS, REPORTS, COMMANDS
+}
+
+typealias ActionCallback = (unit: UnitInformation, action: Action) -> Unit
+
+class DevicesAdapter(
+    private val units: List<UnitInformation>,
+    private val actionCallback: ActionCallback?
+) : RecyclerView.Adapter<DevicesAdapter.ViewHolder>() {
 
     override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): ViewHolder {
         val binding = UnitItemBinding.inflate(LayoutInflater.from(parent.context), parent, false)
@@ -26,6 +35,11 @@ class DevicesAdapter(private val units: List<UnitInformation>) :
             unitSpeed.text = "${unit.position?.speed ?: "--"} Km/h"
             lastAddress.text = unit.position?.address ?: "Unknown location"
             lastDate.text = "yyyy/mm/dd, hh:mm"
+            actionCallback?.let { callback ->
+                detailsButton.setOnClickListener { callback(unit, Action.DETAILS) }
+                reportsButton.setOnClickListener { callback(unit, Action.REPORTS) }
+                commandsButton.setOnClickListener { callback(unit, Action.COMMANDS) }
+            }
         }
     }
 
@@ -38,9 +52,15 @@ class DevicesAdapter(private val units: List<UnitInformation>) :
             }
         }
 
-        fun toggleOptions(shouldShow: Boolean) {
-            binding.itemOptions.visibility = if (shouldShow) View.VISIBLE else View.GONE
-            binding.unitCard.cardElevation = if (shouldShow) 5.0F else 0.0F
+        fun toggleOptions(shouldExpand: Boolean) {
+            val context = binding.root.context
+            binding.unitCard.setCardBackgroundColor(
+                context
+                    .resources
+                    .getColor(if (shouldExpand) R.color.background else R.color.darkBackground)
+            )
+            binding.itemOptions.visibility = if (shouldExpand) View.VISIBLE else View.GONE
+            binding.unitCard.cardElevation = if (shouldExpand) 5.0F else 0.0F
         }
     }
 }
\ No newline at end of file
diff --git a/androidApp/src/main/java/mx/trackermap/TrackerMap/android/devices/DevicesFragment.kt b/androidApp/src/main/java/mx/trackermap/TrackerMap/android/devices/DevicesFragment.kt
index 4ac5d48..a3ed7a8 100644
--- a/androidApp/src/main/java/mx/trackermap/TrackerMap/android/devices/DevicesFragment.kt
+++ b/androidApp/src/main/java/mx/trackermap/TrackerMap/android/devices/DevicesFragment.kt
@@ -10,9 +10,11 @@ import androidx.recyclerview.widget.LinearLayoutManager
 import kotlinx.coroutines.DelicateCoroutinesApi
 import mx.trackermap.TrackerMap.android.databinding.DevicesFragmentBinding
 import mx.trackermap.TrackerMap.android.units.UnitsViewModel
+import mx.trackermap.TrackerMap.client.models.UnitInformation
 import org.koin.androidx.viewmodel.ext.android.viewModel
 
-class DevicesFragment: Fragment() {
+class DevicesFragment : Fragment() {
+
     private var _binding: DevicesFragmentBinding? = null
     private val binding get() = _binding!!
 
@@ -40,15 +42,25 @@ class DevicesFragment: Fragment() {
     }
 
     private fun setupList() {
-        binding.devicesList.layoutManager = LinearLayoutManager(context, LinearLayoutManager.VERTICAL, false)
-        binding.devicesList.adapter = DevicesAdapter(emptyList())
+        binding.devicesList.layoutManager = LinearLayoutManager(
+            context, LinearLayoutManager.VERTICAL,
+            false
+        )
+        binding.devicesList.adapter = DevicesAdapter(emptyList(), null)
     }
 
     @DelicateCoroutinesApi
     private fun setupObservers() {
         unitsViewModel.units.observe(this) { units ->
             Log.d("DevicesFragment", "Success $units")
-            binding.devicesList.swapAdapter(DevicesAdapter(units), false)
+            binding.devicesList.swapAdapter(
+                DevicesAdapter(units, this::itemAction),
+                false
+            )
         }
     }
+
+    private fun itemAction(unit: UnitInformation, action: Action) {
+        Log.d("DevicesFragment", "Action: $action - Unit: $unit")
+    }
 }
\ No newline at end of file
diff --git a/androidApp/src/main/java/mx/trackermap/TrackerMap/android/units/UnitsViewModel.kt b/androidApp/src/main/java/mx/trackermap/TrackerMap/android/units/UnitsViewModel.kt
index 5d7d469..bcb0818 100644
--- a/androidApp/src/main/java/mx/trackermap/TrackerMap/android/units/UnitsViewModel.kt
+++ b/androidApp/src/main/java/mx/trackermap/TrackerMap/android/units/UnitsViewModel.kt
@@ -25,7 +25,7 @@ class UnitsViewModel(
     private val unitsController: UnitsController by inject()
 
     var searchQuery = savedStateHandle.getLiveData("searchQuery", "")
-    var unitsDisplayMode = MutableLiveData(UnitsDisplayMode.MAP)
+    var unitsDisplayMode = MutableLiveData(UnitsDisplayMode.LIST)
     var units = MutableLiveData<List<UnitInformation>>()
 
     init {