From c63afc809743e22bbfd9830c8fb6c0f8147786d6 Mon Sep 17 00:00:00 2001
From: 0xd9a <0xd9a@noreply.codeberg.org>
Date: Sat, 24 Dec 2022 16:20:21 +0530
Subject: [PATCH 1/2] Use 'chip' for description button in compose

---
 .../android/ui/drawer/ComposeAdapter.java     | 12 +++++------
 .../res/layout/compose_attachment_item.xml    | 21 ++++++++-----------
 2 files changed, 15 insertions(+), 18 deletions(-)

diff --git a/app/src/main/java/app/fedilab/android/ui/drawer/ComposeAdapter.java b/app/src/main/java/app/fedilab/android/ui/drawer/ComposeAdapter.java
index 88ae8424..3938b4fc 100644
--- a/app/src/main/java/app/fedilab/android/ui/drawer/ComposeAdapter.java
+++ b/app/src/main/java/app/fedilab/android/ui/drawer/ComposeAdapter.java
@@ -1156,15 +1156,15 @@ public class ComposeAdapter extends RecyclerView.Adapter<RecyclerView.ViewHolder
                     });
                     composeAttachmentItemBinding.preview.setOnClickListener(v -> displayAttachments(holder, position, finalMediaPosition));
                     if (attachment.description == null || attachment.description.trim().isEmpty()) {
-                        composeAttachmentItemBinding.buttonDescription.setIconResource(R.drawable.ic_baseline_warning_24);
+                        composeAttachmentItemBinding.buttonDescription.setChipIconResource(R.drawable.ic_baseline_warning_24);
                         composeAttachmentItemBinding.buttonDescription.setTextColor(ContextCompat.getColor(context, R.color.black));
-                        composeAttachmentItemBinding.buttonDescription.setIconTintResource(R.color.black);
-                        composeAttachmentItemBinding.buttonDescription.setBackgroundTintList(ThemeHelper.getNoDescriptionColorStateList(context));
+                        composeAttachmentItemBinding.buttonDescription.setChipIconTintResource(R.color.black);
+                        composeAttachmentItemBinding.buttonDescription.setChipBackgroundColor(ThemeHelper.getNoDescriptionColorStateList(context));
                     } else {
-                        composeAttachmentItemBinding.buttonDescription.setIconResource(R.drawable.ic_baseline_check_circle_24);
+                        composeAttachmentItemBinding.buttonDescription.setChipIconResource(R.drawable.ic_baseline_check_circle_24);
                         composeAttachmentItemBinding.buttonDescription.setTextColor(ContextCompat.getColor(context, R.color.white));
-                        composeAttachmentItemBinding.buttonDescription.setIconTintResource(R.color.white);
-                        composeAttachmentItemBinding.buttonDescription.setBackgroundTintList(ThemeHelper.getHavingDescriptionColorStateList(context));
+                        composeAttachmentItemBinding.buttonDescription.setChipIconTintResource(R.color.white);
+                        composeAttachmentItemBinding.buttonDescription.setChipBackgroundColor(ThemeHelper.getHavingDescriptionColorStateList(context));
                     }
                     holder.binding.attachmentsList.addView(composeAttachmentItemBinding.getRoot());
                     mediaPosition++;
diff --git a/app/src/main/res/layout/compose_attachment_item.xml b/app/src/main/res/layout/compose_attachment_item.xml
index ebf9c3b0..4beca2ea 100644
--- a/app/src/main/res/layout/compose_attachment_item.xml
+++ b/app/src/main/res/layout/compose_attachment_item.xml
@@ -25,14 +25,14 @@
 
         <androidx.appcompat.widget.AppCompatImageView
             android:id="@+id/button_play"
-            app:srcCompat="@drawable/ic_compose_attachment_play"
             android:layout_width="36dp"
             android:layout_height="36dp"
             android:scaleType="fitCenter"
             app:layout_constraintBottom_toBottomOf="@id/preview"
             app:layout_constraintEnd_toEndOf="@id/preview"
             app:layout_constraintStart_toStartOf="@id/preview"
-            app:layout_constraintTop_toTopOf="@id/preview" />
+            app:layout_constraintTop_toTopOf="@id/preview"
+            app:srcCompat="@drawable/ic_compose_attachment_play" />
 
         <com.google.android.material.button.MaterialButton
             android:id="@+id/edit_preview"
@@ -66,22 +66,19 @@
             app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toBottomOf="@id/preview" />
 
-        <com.google.android.material.button.MaterialButton
+        <com.google.android.material.chip.Chip
             android:id="@+id/button_description"
-            android:layout_width="match_parent"
+            style="@style/Widget.Material3.Chip.Assist.Elevated"
+            android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:layout_marginStart="2dp"
-            android:layout_marginEnd="2dp"
             android:text="@string/description"
-            android:textAlignment="textStart"
             android:textColor="@color/black"
-            app:backgroundTint="@color/no_description"
-            app:icon="@drawable/ic_baseline_warning_24"
-            app:iconGravity="textStart"
-            app:iconTint="@color/black"
+            app:chipBackgroundColor="@color/no_description"
+            app:chipIcon="@drawable/ic_baseline_warning_24"
             app:layout_constraintEnd_toStartOf="@id/button_order_down"
             app:layout_constraintStart_toEndOf="@id/button_order_up"
-            app:layout_constraintTop_toBottomOf="@id/preview" />
+            app:layout_constraintTop_toBottomOf="@id/preview"
+            app:textEndPadding="0dp" />
 
         <com.google.android.material.button.MaterialButton
             android:id="@+id/button_order_down"

From 626e05b6a07072789b05633f1fe61a79a10cd880 Mon Sep 17 00:00:00 2001
From: 0xd9a <0xd9a@noreply.codeberg.org>
Date: Sat, 24 Dec 2022 19:23:16 +0530
Subject: [PATCH 2/2] Update compose UI

---
 app/src/main/res/drawable/bg_compose_panels.xml   | 5 +++++
 app/src/main/res/layout/drawer_status_compose.xml | 9 +++++----
 2 files changed, 10 insertions(+), 4 deletions(-)
 create mode 100644 app/src/main/res/drawable/bg_compose_panels.xml

diff --git a/app/src/main/res/drawable/bg_compose_panels.xml b/app/src/main/res/drawable/bg_compose_panels.xml
new file mode 100644
index 00000000..ada4a62c
--- /dev/null
+++ b/app/src/main/res/drawable/bg_compose_panels.xml
@@ -0,0 +1,5 @@
+<shape xmlns:android="http://schemas.android.com/apk/res/android"
+    android:shape="rectangle">
+    <solid android:color="?colorSurfaceVariant" />
+    <corners android:radius="12dp" />
+</shape>
diff --git a/app/src/main/res/layout/drawer_status_compose.xml b/app/src/main/res/layout/drawer_status_compose.xml
index 5ad3ce8c..ca778b76 100644
--- a/app/src/main/res/layout/drawer_status_compose.xml
+++ b/app/src/main/res/layout/drawer_status_compose.xml
@@ -220,7 +220,7 @@
             android:id="@+id/attachment_choices_panel"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:background="?colorSurfaceVariant"
+            android:background="@drawable/bg_compose_panels"
             android:padding="6dp"
             android:visibility="gone"
             app:constraint_referenced_ids="button_attach_image,button_attach_audio,button_attach_video,button_close_attachment_panel,button_poll,button_attach_manual"
@@ -274,14 +274,15 @@
         <androidx.appcompat.widget.LinearLayoutCompat
             android:id="@+id/visibility_panel"
             android:layout_width="match_parent"
-            android:layout_height="0dp"
-            android:background="?colorSurfaceVariant"
+            android:layout_height="wrap_content"
+            android:background="@drawable/bg_compose_panels"
             android:gravity="bottom"
             android:orientation="vertical"
             android:padding="6dp"
             android:visibility="gone"
             app:layout_constraintBottom_toBottomOf="parent"
-            app:layout_constraintTop_toTopOf="parent">
+            app:layout_constraintTop_toTopOf="parent"
+            app:layout_constraintVertical_bias="1">
 
             <com.google.android.material.button.MaterialButtonToggleGroup
                 android:layout_width="match_parent"