From 15576b6c39cded5d32eb07254dac981af52f58c7 Mon Sep 17 00:00:00 2001
From: Peter Vacho <p_vacho@utb.cz>
Date: Sat, 4 Jan 2025 22:32:48 +0100
Subject: [PATCH] feat(categories): Improve category item layout

---
 .../neat_calendar/adapters/CategoryAdapter.kt |  1 -
 app/src/main/res/layout/item_category.xml     | 89 ++++++++++---------
 2 files changed, 49 insertions(+), 41 deletions(-)

diff --git a/app/src/main/java/com/p_vacho/neat_calendar/adapters/CategoryAdapter.kt b/app/src/main/java/com/p_vacho/neat_calendar/adapters/CategoryAdapter.kt
index 6273888..4bd0a70 100644
--- a/app/src/main/java/com/p_vacho/neat_calendar/adapters/CategoryAdapter.kt
+++ b/app/src/main/java/com/p_vacho/neat_calendar/adapters/CategoryAdapter.kt
@@ -1,6 +1,5 @@
 package com.p_vacho.neat_calendar.adapters
 
-import android.graphics.Color
 import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup
diff --git a/app/src/main/res/layout/item_category.xml b/app/src/main/res/layout/item_category.xml
index eb1aa79..914a9f5 100644
--- a/app/src/main/res/layout/item_category.xml
+++ b/app/src/main/res/layout/item_category.xml
@@ -1,50 +1,59 @@
 <?xml version="1.0" encoding="utf-8"?>
-<androidx.constraintlayout.widget.ConstraintLayout
+<androidx.cardview.widget.CardView
     xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:app="http://schemas.android.com/apk/res-auto"
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
-    android:padding="8dp">
+    android:layout_margin="8dp"
+    app:cardCornerRadius="8dp"
+    app:cardElevation="4dp"
+    app:cardBackgroundColor="?android:attr/colorBackground">
 
-    <!-- Color Indicator -->
-    <View
-        android:id="@+id/colorIndicator"
-        android:layout_width="8dp"
-        android:layout_height="match_parent"
-        android:background="@android:color/holo_blue_dark"
-        app:layout_constraintTop_toTopOf="parent"
-        app:layout_constraintBottom_toBottomOf="parent"
-        app:layout_constraintStart_toStartOf="parent" />
-
-    <!-- Category Name -->
-    <TextView
-        android:id="@+id/tvCategoryName"
-        android:layout_width="0dp"
+    <androidx.constraintlayout.widget.ConstraintLayout
+        android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        android:textSize="16sp"
-        android:textColor="?android:attr/textColorPrimary"
-        android:paddingStart="16dp"
-        android:paddingEnd="8dp"
-        android:ellipsize="end"
-        android:maxLines="1"
-        tools:text="Work"
-        app:layout_constraintTop_toTopOf="parent"
-        app:layout_constraintBottom_toBottomOf="parent"
-        app:layout_constraintStart_toEndOf="@id/colorIndicator"
-        app:layout_constraintEnd_toStartOf="@id/btnDeleteCategory" />
+        android:padding="8dp">
 
-    <!-- Delete Button -->
-    <ImageButton
-        android:id="@+id/btnDeleteCategory"
-        android:layout_width="40dp"
-        android:layout_height="40dp"
-        android:background="?attr/selectableItemBackgroundBorderless"
-        android:src="@drawable/ic_trashbin"
-        android:contentDescription="@string/delete_category"
-        app:tint="?android:attr/textColorSecondary"
-        app:layout_constraintTop_toTopOf="parent"
-        app:layout_constraintBottom_toBottomOf="parent"
-        app:layout_constraintEnd_toEndOf="parent" />
+        <!-- Vertical Color Indicator -->
+        <View
+            android:id="@+id/colorIndicator"
+            android:layout_width="8dp"
+            android:layout_height="0dp"
+            android:background="@android:color/holo_blue_dark"
+            app:layout_constraintTop_toTopOf="parent"
+            app:layout_constraintBottom_toBottomOf="parent"
+            app:layout_constraintStart_toStartOf="parent" />
 
-</androidx.constraintlayout.widget.ConstraintLayout>
+        <!-- Category Name -->
+        <TextView
+            android:id="@+id/tvCategoryName"
+            android:layout_width="0dp"
+            android:layout_height="wrap_content"
+            android:textSize="16sp"
+            android:textColor="?android:attr/textColorPrimary"
+            android:paddingStart="16dp"
+            android:paddingEnd="8dp"
+            android:ellipsize="end"
+            android:maxLines="1"
+            tools:text="Work"
+            app:layout_constraintTop_toTopOf="parent"
+            app:layout_constraintBottom_toBottomOf="parent"
+            app:layout_constraintStart_toEndOf="@id/colorIndicator"
+            app:layout_constraintEnd_toStartOf="@id/btnDeleteCategory" />
+
+        <!-- Delete Button -->
+        <ImageButton
+            android:id="@+id/btnDeleteCategory"
+            android:layout_width="40dp"
+            android:layout_height="40dp"
+            android:background="?attr/selectableItemBackgroundBorderless"
+            android:src="@drawable/ic_trashbin"
+            android:contentDescription="@string/delete_category"
+            app:tint="?android:attr/textColorSecondary"
+            app:layout_constraintTop_toTopOf="parent"
+            app:layout_constraintBottom_toBottomOf="parent"
+            app:layout_constraintEnd_toEndOf="parent" />
+
+    </androidx.constraintlayout.widget.ConstraintLayout>
+</androidx.cardview.widget.CardView>