Merge pull request #110 from Stack-Knowledge/feature/109_goods_image_click_check

🔀 :: (#109) Goods Image Click Check

Author: yeongun130

feature/shop/src/main/java/com/stackknowledge/shop/component/GoodsItem.kt

@@ -11,12 +11,14 @@ import androidx.compose.foundation.layout.fillMaxWidth
 import androidx.compose.foundation.layout.height
 import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.layout.width
+import androidx.compose.foundation.shape.RoundedCornerShape
 import androidx.compose.material3.Text
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
 import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
+import androidx.compose.ui.draw.clip
 import androidx.compose.ui.layout.ContentScale
 import androidx.compose.ui.res.painterResource
 import androidx.compose.ui.res.stringResource
@@ -47,7 +49,18 @@ fun GoodsItem(
             modifier = modifier
                 .background(color = colors.WHITE)
         ) {
-            Box {
+            Box(
+                modifier = modifier
+                    .clip(shape = RoundedCornerShape(10.dp))
+                    .clickable {
+                        if (checked.value) {
+                            onItemCheckButtonClick(itemData)
+                        } else {
+                            onItemUnCheckButtonClick(itemData)
+                        }
+                        checked.value = !checked.value
+                    }
+            ) {
                 val imagePainter = if (itemData.image.isEmpty()) {
                     painterResource(id = R.drawable.goods_image)
                 } else {
@@ -58,7 +71,9 @@ fun GoodsItem(
                     painter = imagePainter,
                     contentDescription = "Goods Image",
                     contentScale = ContentScale.Crop,
-                    modifier = Modifier.height(150.dp).fillMaxWidth()
+                    modifier = Modifier
+                        .height(150.dp)
+                        .fillMaxWidth()
                 )
 
                 Box(
@@ -70,15 +85,7 @@ fun GoodsItem(
                         contentDescription = null,
                         modifier = Modifier
                             .width(16.dp)
-                            .height(16.dp)
-                            .clickable {
-                                if (checked.value) {
-                                    onItemCheckButtonClick(itemData)
-                                } else {
-                                    onItemUnCheckButtonClick(itemData)
-                                }
-                                checked.value = !checked.value
-                            },
+                            .height(16.dp),
                         contentScale = ContentScale.Crop
                     )
                 }