Fix button overflow: equal-weight layout; move tag loading to background thread

Author: adegard

app/src/main/java/com/degard/imagecompressor/MainActivity.kt

@@ -170,11 +170,22 @@ class MainActivity : AppCompatActivity() {
         val images = mutableListOf<GalleryAdapter.ImageEntry>()
         var imgIndex = 0
         entries.filter { !it.isDirectory }.forEach {
-            val hasTag = TagManager.hasTag(this, Uri.parse(it.uri))
-            images.add(GalleryAdapter.ImageEntry(Uri.parse(it.uri), it.name, imgIndex++, hasTag))
+            images.add(GalleryAdapter.ImageEntry(Uri.parse(it.uri), it.name, imgIndex++, hasTag = false))
         }
 
         adapter.submitData(folders, images)
+
+        Thread {
+            val tagMap = mutableMapOf<Int, Boolean>()
+            for (img in images) {
+                tagMap[img.index] = TagManager.hasTag(this, img.uri)
+            }
+            runOnUiThread {
+                for ((idx, hasTag) in tagMap) {
+                    adapter.updateHasTag(idx, hasTag)
+                }
+            }
+        }.start()
     }
 
     private fun queryChildren(treeUri: Uri, parentDocId: String): List<FolderCache.CachedEntry> {

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

@@ -49,48 +49,60 @@
 
         <com.google.android.material.button.MaterialButton
             android:id="@+id/btnDelete"
-            android:layout_width="wrap_content"
+            android:layout_width="0dp"
             android:layout_height="40dp"
+            android:layout_weight="1"
+            android:layout_marginEnd="4dp"
             android:text="@string/delete"
             android:textColor="@android:color/white"
+            android:textSize="11sp"
             app:icon="@drawable/ic_delete"
             app:iconTint="@android:color/white"
+            app:iconSize="18dp"
             app:backgroundTint="#AAE53935"
             style="@style/Widget.Material3.Button" />
 
         <com.google.android.material.button.MaterialButton
             android:id="@+id/btnRotate"
-            android:layout_width="wrap_content"
+            android:layout_width="0dp"
             android:layout_height="40dp"
-            android:layout_marginStart="8dp"
+            android:layout_weight="1"
+            android:layout_marginEnd="4dp"
             android:text="@string/rotate"
             android:textColor="@android:color/white"
+            android:textSize="11sp"
             app:icon="@drawable/ic_rotate"
             app:iconTint="@android:color/white"
+            app:iconSize="18dp"
             app:backgroundTint="#AA455A64"
             style="@style/Widget.Material3.Button" />
 
         <com.google.android.material.button.MaterialButton
             android:id="@+id/btnShare"
-            android:layout_width="wrap_content"
+            android:layout_width="0dp"
             android:layout_height="40dp"
-            android:layout_marginStart="8dp"
+            android:layout_weight="1"
+            android:layout_marginEnd="4dp"
             android:text="@string/share"
             android:textColor="@android:color/white"
+            android:textSize="11sp"
             app:icon="@drawable/ic_share"
             app:iconTint="@android:color/white"
+            app:iconSize="18dp"
             app:backgroundTint="#AA33B5E5"
             style="@style/Widget.Material3.Button" />
 
         <com.google.android.material.button.MaterialButton
             android:id="@+id/btnTag"
-            android:layout_width="wrap_content"
+            android:layout_width="0dp"
             android:layout_height="40dp"
-            android:layout_marginStart="8dp"
+            android:layout_weight="1"
             android:text="@string/tag"
             android:textColor="@android:color/white"
+            android:textSize="11sp"
             app:icon="@drawable/ic_tag"
             app:iconTint="@android:color/white"
+            app:iconSize="18dp"
             app:backgroundTint="#AAFFA000"
             style="@style/Widget.Material3.Button" />