Offset icon buttons instead of outline text button

Lambada10 · Dec 9, 2024 · 6d1d99d · 6d1d99d
1 parent 789d961
commit 6d1d99d
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 10 deletions.
diff --git a/...c/main/java/pl/lambada/songsync/ui/screens/lyricsFetch/components/LyricsSuccessContent.kt b/...c/main/java/pl/lambada/songsync/ui/screens/lyricsFetch/components/LyricsSuccessContent.kt
@@ -20,11 +20,14 @@ import androidx.compose.foundation.layout.width
 import androidx.compose.foundation.shape.RoundedCornerShape
 import androidx.compose.foundation.text.selection.SelectionContainer
 import androidx.compose.material.icons.Icons
+import androidx.compose.material.icons.filled.Add
 import androidx.compose.material.icons.filled.Exposure
+import androidx.compose.material.icons.filled.Remove
 import androidx.compose.material.icons.outlined.ContentCopy
 import androidx.compose.material3.Button
 import androidx.compose.material3.HorizontalDivider
 import androidx.compose.material3.Icon
+import androidx.compose.material3.IconButton
 import androidx.compose.material3.LocalContentColor
 import androidx.compose.material3.MaterialTheme
 import androidx.compose.material3.OutlinedButton
@@ -75,37 +78,43 @@ fun LyricsSuccessContent(
                 modifier = Modifier.padding(start = 6.dp)
             )
             Spacer(modifier = Modifier.weight(1f))
-            OutlinedButton(
+            IconButton(
                 onClick = { /* handled by repeatingClickable */ },
                 modifier = Modifier
-                    .clip(RoundedCornerShape(20.dp)) // otherwise square ripple
                     .repeatingClickable(
                         interactionSource = remember { MutableInteractionSource() },
                         enabled = true,
                         maxDelayMillis = 500,
                         onClick = { onSetOffset(offset - 100) }
                     )
             ) {
-                Text(text = "-0.1s")
+                Icon(
+                    imageVector = Icons.Default.Remove,
+                    contentDescription = null,
+                    modifier = Modifier.size(20.dp)
+                )
             }
             Spacer(modifier = Modifier.width(10.dp))
             Text(
                 text = (if (offset >= 0) "+" else "") +
                     "${offset / 1000.0}s",
             )
             Spacer(modifier = Modifier.width(10.dp))
-            OutlinedButton(
+            IconButton(
                 onClick = { /* handled by repeatingClickable */ },
                 modifier = Modifier
-                    .clip(RoundedCornerShape(20.dp)) // otherwise square ripple
                     .repeatingClickable(
                         interactionSource = remember { MutableInteractionSource() },
                         enabled = true,
                         maxDelayMillis = 500,
                         onClick = { onSetOffset(offset + 100) }
                     )
             ) {
-                Text(text = "+0.1s")
+                Icon(
+                    imageVector = Icons.Default.Add,
+                    contentDescription = null,
+                    modifier = Modifier.size(20.dp)
+                )
             }
         }
 

diff --git a/app/src/main/java/pl/lambada/songsync/util/ext/ComposeExt.kt b/app/src/main/java/pl/lambada/songsync/util/ext/ComposeExt.kt
@@ -95,9 +95,6 @@ fun Modifier.repeatingClickable(
                     }
                 }
             }
-        }.indication(
-            interactionSource = interactionSource,
-            indication = ripple()
-        )
+        }
     }
 )