Update ItemEditViewModel.kt

Feedback Update ItemDetailsScreen.kt

Author: android-dev-lxl

app/src/main/java/com/example/inventory/ui/home/HomeScreen.kt

@@ -95,7 +95,7 @@ fun HomeScreen(
  HomeBody(
  itemList = listOf(),
  onItemClick = navigateToItemUpdate,
- modifier = modifier
+ modifier = Modifier
  .padding(innerPadding)
  .fillMaxSize()
  )
@@ -145,7 +145,8 @@ private fun InventoryItem(
  item: Item, modifier: Modifier = Modifier
 ) {
  Card(
- modifier = modifier, elevation = CardDefaults.cardElevation(defaultElevation = 2.dp)
+ modifier = modifier,
+ elevation = CardDefaults.cardElevation(defaultElevation = 2.dp)
  ) {
  Column(
  modifier = Modifier.padding(dimensionResource(id = R.dimen.padding_large)),

app/src/main/java/com/example/inventory/ui/item/ItemDetailsScreen.kt

@@ -67,25 +67,26 @@ fun ItemDetailsScreen(
  navigateBack: () -> Unit,
  modifier: Modifier = Modifier
 ) {
- Scaffold(topBar = {
- InventoryTopAppBar(
- title = stringResource(ItemDetailsDestination.titleRes),
- canNavigateBack = true,
- navigateUp = navigateBack
- )
- }, floatingActionButton = {
- FloatingActionButton(
- onClick = { navigateToEditItem(0) },
- shape = MaterialTheme.shapes.medium,
- modifier = Modifier.padding(dimensionResource(id = R.dimen.padding_large))
-
- ) {
- Icon(
- imageVector = Icons.Default.Edit,
- contentDescription = stringResource(R.string.edit_item_title),
+ Scaffold(
+ topBar = {
+ InventoryTopAppBar(
+ title = stringResource(ItemDetailsDestination.titleRes),
+ canNavigateBack = true,
+ navigateUp = navigateBack
  )
- }
- }, modifier = modifier
+ }, floatingActionButton = {
+ FloatingActionButton(
+ onClick = { navigateToEditItem(0) },
+ shape = MaterialTheme.shapes.medium,
+ modifier = Modifier.padding(dimensionResource(id = R.dimen.padding_large))
+
+ ) {
+ Icon(
+ imageVector = Icons.Default.Edit,
+ contentDescription = stringResource(R.string.edit_item_title),
+ )
+ }
+ }, modifier = modifier
  ) { innerPadding ->
  ItemDetailsBody(
  itemDetailsUiState = ItemDetailsUiState(),
@@ -112,7 +113,8 @@ private fun ItemDetailsBody(
  var deleteConfirmationRequired by rememberSaveable { mutableStateOf(false) }
 
  ItemDetails(
- item = itemDetailsUiState.itemDetails.toItem(), modifier = Modifier.fillMaxWidth()
+ item = itemDetailsUiState.itemDetails.toItem(),
+ modifier = Modifier.fillMaxWidth()
  )
  Button(
  onClick = onSellItem,
@@ -147,7 +149,8 @@ fun ItemDetails(
  item: Item, modifier: Modifier = Modifier
 ) {
  Card(
- modifier = modifier, colors = CardDefaults.cardColors(
+ modifier = modifier,
+ colors = CardDefaults.cardColors(
  containerColor = MaterialTheme.colorScheme.primaryContainer,
  contentColor = MaterialTheme.colorScheme.onPrimaryContainer
  )
@@ -156,40 +159,32 @@ fun ItemDetails(
  modifier = Modifier
  .fillMaxWidth()
  .padding(dimensionResource(id = R.dimen.padding_medium)),
- verticalArrangement = Arrangement.spacedBy(dimensionResource(id = R.dimen.padding_medium))
+ verticalArrangement = Arrangement.spacedBy(
+ dimensionResource(id = R.dimen.padding_medium)
+ )
  ) {
  ItemDetailsRow(
  labelResID = R.string.item,
  itemDetail = item.name,
  modifier = Modifier.padding(
- horizontal = dimensionResource(
- id = R.dimen
- .padding_medium
- )
+ horizontal = dimensionResource(id = R.dimen.padding_medium)
  )
  )
  ItemDetailsRow(
  labelResID = R.string.quantity_in_stock,
  itemDetail = item.quantity.toString(),
  modifier = Modifier.padding(
- horizontal = dimensionResource(
- id = R.dimen
- .padding_medium
- )
+ horizontal = dimensionResource(id = R.dimen.padding_medium)
  )
  )
  ItemDetailsRow(
  labelResID = R.string.price,
  itemDetail = item.formatedPrice(),
  modifier = Modifier.padding(
- horizontal = dimensionResource(
- id = R.dimen
- .padding_medium
- )
+ horizontal = dimensionResource(id = R.dimen.padding_medium)
  )
  )
  }
-
  }
 }
 
@@ -206,7 +201,9 @@ private fun ItemDetailsRow(
 
 @Composable
 private fun DeleteConfirmationDialog(
- onDeleteConfirm: () -> Unit, onDeleteCancel: () -> Unit, modifier: Modifier = Modifier
+ onDeleteConfirm: () -> Unit,
+ onDeleteCancel: () -> Unit,
+ modifier: Modifier = Modifier
 ) {
  AlertDialog(onDismissRequest = { /* Do nothing */ },
  title = { Text(stringResource(R.string.attention)) },

app/src/main/java/com/example/inventory/ui/item/ItemEntryScreen.kt

@@ -85,9 +85,9 @@ fun ItemEntryBody(
  modifier: Modifier = Modifier
 ) {
  Column(
- modifier = modifier.padding(dimensionResource(id = R.dimen.padding_medium)),
- verticalArrangement = Arrangement.spacedBy(dimensionResource(id = R.dimen.padding_large))
- ) {
+ verticalArrangement = Arrangement.spacedBy(dimensionResource(id = R.dimen.padding_large)),
+ modifier = modifier.padding(dimensionResource(id = R.dimen.padding_medium))
+  ) {
  ItemInputForm(
  itemDetails = itemUiState.itemDetails,
  onValueChange = onItemValueChange,