Projets/SafeBite
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: replace LazyVerticalGrid with Column/Row layout in AllergenSelectionGrid for improved performance and layout control

Browse Source
This commit is contained in:
Bruno Charest 2026-04-26 12:42:51 -04:00
parent 1656b189f4
commit a9eb582c93
1 changed files with 28 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

47
app/src/main/java/com/safebite/app/presentation/common/components/AllergenGrid.kt
View File

@ -7,12 +7,10 @@ import androidx.compose.foundation.layout.Arrangement
import androidx.compose.foundation.layout.Box
import androidx.compose.foundation.layout.Column
import androidx.compose.foundation.layout.Row
import androidx.compose.foundation.layout.Spacer
import androidx.compose.foundation.layout.fillMaxWidth
import androidx.compose.foundation.layout.padding
import androidx.compose.foundation.layout.size
import androidx.compose.foundation.lazy.grid.GridCells
import androidx.compose.foundation.lazy.grid.LazyVerticalGrid
import androidx.compose.foundation.lazy.grid.items
import androidx.compose.foundation.shape.RoundedCornerShape
import androidx.compose.material3.Card
import androidx.compose.material3.CardDefaults
@ -71,27 +69,38 @@ fun AllergenSelectionGrid(
modifier: Modifier = Modifier
) {
val dimens = LocalDimens.current
val columns = 3
val rows = AllergenType.entries.toList().chunked(columns)
LazyVerticalGrid(
columns = GridCells.Fixed(3),
Column(
modifier = modifier.fillMaxWidth(),
horizontalArrangement = Arrangement.spacedBy(dimens.spacingSm),
verticalArrangement = Arrangement.spacedBy(dimens.spacingSm)
) {
items(AllergenType.entries.toList()) { allergen ->
val currentLevel = selectedAllergens[allergen] ?: AllergenLevel.NONE
AllergenSelectionChip(
allergen = allergen,
level = currentLevel,
onClick = {
val nextLevel = when (currentLevel) {
AllergenLevel.NONE -> AllergenLevel.TRACE
AllergenLevel.TRACE -> AllergenLevel.SEVERE
AllergenLevel.SEVERE -> AllergenLevel.NONE
}
onLevelChanged(allergen, nextLevel)
rows.forEach { row ->
Row(
modifier = Modifier.fillMaxWidth(),
horizontalArrangement = Arrangement.spacedBy(dimens.spacingSm)
) {
row.forEach { allergen ->
val currentLevel = selectedAllergens[allergen] ?: AllergenLevel.NONE
AllergenSelectionChip(
modifier = Modifier.weight(1f),
allergen = allergen,
level = currentLevel,
onClick = {
val nextLevel = when (currentLevel) {
AllergenLevel.NONE -> AllergenLevel.TRACE
AllergenLevel.TRACE -> AllergenLevel.SEVERE
AllergenLevel.SEVERE -> AllergenLevel.NONE
}
onLevelChanged(allergen, nextLevel)
}
)
}
)
repeat(columns - row.size) {
Spacer(modifier = Modifier.weight(1f))
}
}
}
}
}