From 9a06fb4d75697f89f6eb409ab61c513972ca10c8 Mon Sep 17 00:00:00 2001
From: Sad Ellie <openthiswhitedoor@gmail.com>
Date: Tue, 23 May 2023 10:22:00 +0300
Subject: [PATCH] Squashable unit selection button

---
 .../core/ui/common/ModifierExtensions.kt      |  4 +++
 .../unitto/core/ui/common/UnittoButton.kt     |  6 +++--
 .../components/UnitSelectionButton.kt         | 25 +++----------------
 3 files changed, 11 insertions(+), 24 deletions(-)

diff --git a/core/ui/src/main/java/com/sadellie/unitto/core/ui/common/ModifierExtensions.kt b/core/ui/src/main/java/com/sadellie/unitto/core/ui/common/ModifierExtensions.kt
index bca4b685..6bcd4add 100644
--- a/core/ui/src/main/java/com/sadellie/unitto/core/ui/common/ModifierExtensions.kt
+++ b/core/ui/src/main/java/com/sadellie/unitto/core/ui/common/ModifierExtensions.kt
@@ -37,6 +37,7 @@ import androidx.compose.ui.unit.Dp
 fun Modifier.squashable(
     onClick: () -> Unit = {},
     onLongClick: (() -> Unit)? = null,
+    enabled: Boolean = true,
     interactionSource: MutableInteractionSource,
     cornerRadiusRange: IntRange,
     role: Role = Role.Button,
@@ -55,12 +56,14 @@ fun Modifier.squashable(
             interactionSource = interactionSource,
             indication = rememberRipple(),
             role = role,
+            enabled = enabled
         )
 }
 
 fun Modifier.squashable(
     onClick: () -> Unit = {},
     onLongClick: (() -> Unit)? = null,
+    enabled: Boolean = true,
     interactionSource: MutableInteractionSource,
     cornerRadiusRange: ClosedRange<Dp>,
     role: Role = Role.Button,
@@ -79,5 +82,6 @@ fun Modifier.squashable(
             interactionSource = interactionSource,
             indication = rememberRipple(),
             role = role,
+            enabled = enabled
         )
 }
diff --git a/core/ui/src/main/java/com/sadellie/unitto/core/ui/common/UnittoButton.kt b/core/ui/src/main/java/com/sadellie/unitto/core/ui/common/UnittoButton.kt
index 8416a49e..5890bc65 100644
--- a/core/ui/src/main/java/com/sadellie/unitto/core/ui/common/UnittoButton.kt
+++ b/core/ui/src/main/java/com/sadellie/unitto/core/ui/common/UnittoButton.kt
@@ -44,6 +44,7 @@ fun UnittoButton(
     modifier: Modifier = Modifier,
     onClick: () -> Unit,
     onLongClick: (() -> Unit)? = null,
+    enabled: Boolean = true,
     containerColor: Color,
     contentColor: Color = contentColorFor(containerColor),
     border: BorderStroke? = null,
@@ -56,11 +57,12 @@ fun UnittoButton(
             onClick = onClick,
             onLongClick = onLongClick,
             interactionSource = interactionSource,
-            cornerRadiusRange = 30..50
+            cornerRadiusRange = 30..50,
+            enabled = enabled
         ),
         color = containerColor,
         contentColor = contentColor,
-        border = border
+        border = border,
     ) {
         CompositionLocalProvider(LocalContentColor provides contentColor) {
             ProvideTextStyle(value = MaterialTheme.typography.labelLarge) {
diff --git a/feature/converter/src/main/java/com/sadellie/unitto/feature/converter/components/UnitSelectionButton.kt b/feature/converter/src/main/java/com/sadellie/unitto/feature/converter/components/UnitSelectionButton.kt
index 65a03503..40097f47 100644
--- a/feature/converter/src/main/java/com/sadellie/unitto/feature/converter/components/UnitSelectionButton.kt
+++ b/feature/converter/src/main/java/com/sadellie/unitto/feature/converter/components/UnitSelectionButton.kt
@@ -21,30 +21,22 @@ package com.sadellie.unitto.feature.converter.components
 import android.annotation.SuppressLint
 import androidx.compose.animation.AnimatedContent
 import androidx.compose.animation.SizeTransform
-import androidx.compose.animation.core.FastOutSlowInEasing
-import androidx.compose.animation.core.animateIntAsState
-import androidx.compose.animation.core.tween
 import androidx.compose.animation.fadeIn
 import androidx.compose.animation.fadeOut
 import androidx.compose.animation.slideInVertically
 import androidx.compose.animation.slideOutVertically
 import androidx.compose.animation.togetherWith
-import androidx.compose.foundation.interaction.MutableInteractionSource
-import androidx.compose.foundation.interaction.collectIsPressedAsState
 import androidx.compose.foundation.layout.PaddingValues
-import androidx.compose.foundation.shape.RoundedCornerShape
 import androidx.compose.material3.Button
-import androidx.compose.material3.ButtonDefaults
 import androidx.compose.material3.MaterialTheme
 import androidx.compose.material3.Text
 import androidx.compose.runtime.Composable
-import androidx.compose.runtime.getValue
-import androidx.compose.runtime.remember
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.text.style.TextOverflow
 import androidx.compose.ui.unit.dp
 import com.sadellie.unitto.core.base.R
+import com.sadellie.unitto.core.ui.common.UnittoButton
 
 /**
  * Button to select a unit
@@ -60,23 +52,12 @@ internal fun UnitSelectionButton(
     onClick: () -> Unit = {},
     label: Int?,
 ) {
-    val interactionSource = remember { MutableInteractionSource() }
-    val isPressed by interactionSource.collectIsPressedAsState()
-    val cornerRadius: Int by animateIntAsState(
-        targetValue = if (isPressed) 30 else 50,
-        animationSpec = tween(easing = FastOutSlowInEasing),
-    )
-
-    Button(
+    UnittoButton(
         modifier = modifier,
-        shape = RoundedCornerShape(cornerRadius),
         onClick = onClick,
         enabled = label != null,
-        colors = ButtonDefaults.buttonColors(
-            containerColor = MaterialTheme.colorScheme.primaryContainer
-        ),
+        containerColor = MaterialTheme.colorScheme.primaryContainer,
         contentPadding = PaddingValues(vertical = 16.dp, horizontal = 8.dp),
-        interactionSource = interactionSource
     ) {
         AnimatedContent(
             targetState = label ?: 0,