♻️ Adjust grid size and simplify onClick function

Grid size in the ResultPanel.kt has been increased to 320.dp to enhance user interaction by improving visibility. Also, in RegexMatchOption.kt, the call 'Modifier.onClick' has been swapped to 'Modifier.clickable' for both ease of understanding and consistency across the code.
CXwudi · Dec 1, 2023 · 9e440de · 9e440de
1 parent 5066871
commit 9e440de
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/songfinder-app/src/main/kotlin/mikufan/cx/songfinder/ui/component/main/RegexMatchOption.kt b/songfinder-app/src/main/kotlin/mikufan/cx/songfinder/ui/component/main/RegexMatchOption.kt
@@ -1,9 +1,9 @@
 package mikufan.cx.songfinder.ui.component.main
 
 import androidx.compose.foundation.ExperimentalFoundationApi
+import androidx.compose.foundation.clickable
 import androidx.compose.foundation.layout.Arrangement
 import androidx.compose.foundation.layout.Row
-import androidx.compose.foundation.onClick
 import androidx.compose.material3.MaterialTheme
 import androidx.compose.material3.RadioButton
 import androidx.compose.material3.Text
@@ -71,6 +71,6 @@ fun RegexMatchOptionButton(
   )
   Text(
     text = renderedOption.displayName,
-    modifier = Modifier.onClick { scope.launch { onOptionSet(renderedOption) } }
+    modifier = Modifier.clickable { scope.launch { onOptionSet(renderedOption) } }
   )
 }
diff --git a/songfinder-app/src/main/kotlin/mikufan/cx/songfinder/ui/component/main/ResultPanel.kt b/songfinder-app/src/main/kotlin/mikufan/cx/songfinder/ui/component/main/ResultPanel.kt
@@ -80,7 +80,7 @@ fun ResultPanelGrid(
     modifier = modifier.padding(horizontal = MaterialTheme.spacing.padding),
   ) {
     LazyVerticalGrid(
-      columns = GridCells.Adaptive(256.dp),
+      columns = GridCells.Adaptive(320.dp),
       horizontalArrangement = Arrangement.spacedBy(MaterialTheme.spacing.spacingSmall),
       verticalArrangement = Arrangement.spacedBy(MaterialTheme.spacing.spacingSmall),
 //      modifier = modifier.padding(end = MaterialTheme.spacing.paddingLarge),