chore: Simplify some Spacers code

Infomaniak · Nov 19, 2024 · bb2852d · bb2852d
1 parent 2f14f6f
commit bb2852d
Show file tree

Hide file tree

Showing 24 changed files with 58 additions and 58 deletions.
diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/components/BottomSheetItem.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/components/BottomSheetItem.kt
@@ -58,7 +58,7 @@ fun BottomSheetItem(imageVector: ImageVector, @StringRes titleRes: Int, onClick:
                 contentDescription = null,
                 tint = SwissTransferTheme.materialColors.primary,
             )
-            Spacer(modifier = Modifier.width(Margin.Large))
+            Spacer(Modifier.width(Margin.Large))
             Text(
                 text = stringResource(id = titleRes),
                 style = SwissTransferTheme.typography.bodyRegular,

diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/components/BrandTopAppBar.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/components/BrandTopAppBar.kt
@@ -46,11 +46,11 @@ fun BrandTopAppBar() {
         title = {
             Row(verticalAlignment = Alignment.CenterVertically) {
                 Image(imageVector = AppIllus.LogoInfomaniak, contentDescription = null)
-                Spacer(modifier = Modifier.width(Margin.Medium))
+                Spacer(Modifier.width(Margin.Medium))
                 VerticalDivider(modifier = Modifier.height(Margin.Large), color = SwissTransferTheme.colors.toolbarTextColor)
-                Spacer(modifier = Modifier.width(Margin.Medium))
+                Spacer(Modifier.width(Margin.Medium))
                 Image(imageVector = AppIllus.LogoSwissTransfer, contentDescription = null)
-                Spacer(modifier = Modifier.width(Margin.Mini))
+                Spacer(Modifier.width(Margin.Mini))
                 Text(text = stringResource(id = R.string.appName), color = SwissTransferTheme.colors.toolbarTextColor)
             }
         },

diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/components/Buttons.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/components/Buttons.kt
@@ -152,7 +152,7 @@ fun KeepButtonSize(imageVector: ImageVector?, titleRes: Int, content: @Composabl
 private fun ButtonTextContent(imageVector: ImageVector?, titleRes: Int) {
     imageVector?.let {
         Icon(modifier = Modifier.size(Margin.Medium), imageVector = it, contentDescription = null)
-        Spacer(modifier = Modifier.width(Margin.Mini))
+        Spacer(Modifier.width(Margin.Mini))
     }
     Text(text = stringResource(id = titleRes), style = SwissTransferTheme.typography.bodyMedium)
 }
@@ -208,7 +208,7 @@ private fun LargeButtonPreview() {
                 ButtonType.entries.forEach {
                     Row {
                         LargeButton(titleRes = R.string.appName, style = it, onClick = {}, imageVector = AppIcons.Add)
-                        Spacer(modifier = Modifier.width(Margin.Mini))
+                        Spacer(Modifier.width(Margin.Mini))
                         LargeButton(
                             titleRes = R.string.appName,
                             style = it,
@@ -217,10 +217,10 @@ private fun LargeButtonPreview() {
                             imageVector = AppIcons.Add
                         )
 
-                        Spacer(modifier = Modifier.width(Margin.Mini))
+                        Spacer(Modifier.width(Margin.Mini))
 
                         SmallButton(titleRes = R.string.appName, style = it, imageVector = AppIcons.Add, onClick = {})
-                        Spacer(modifier = Modifier.width(Margin.Mini))
+                        Spacer(Modifier.width(Margin.Mini))
                         SmallButton(
                             titleRes = R.string.appName,
                             style = it,
@@ -229,7 +229,7 @@ private fun LargeButtonPreview() {
                             onClick = {}
                         )
                     }
-                    Spacer(modifier = Modifier.height(Margin.Medium))
+                    Spacer(Modifier.height(Margin.Medium))
                 }
             }
         }

diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/components/DoubleButtonCombo.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/components/DoubleButtonCombo.kt
@@ -66,7 +66,7 @@ private fun VerticallyStackedButtons(
                 .padding(horizontal = Margin.Medium),
         )
 
-        Spacer(modifier = Modifier.height(Margin.Medium))
+        Spacer(Modifier.height(Margin.Medium))
 
         bottomButton(
             Modifier

diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/components/FileItem.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/components/FileItem.kt
@@ -146,7 +146,7 @@ private fun FileItemPreview(@PreviewParameter(FileUiListPreviewParameter::class)
                     onRemove = {},
                 )
 
-                Spacer(modifier = Modifier.height(Margin.Medium))
+                Spacer(Modifier.height(Margin.Medium))
 
                 val imageFile = files[1]
                 FileItem(

diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/components/NewTransferFab.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/components/NewTransferFab.kt
@@ -74,9 +74,9 @@ private fun NewTransferFabPreview() {
         Surface {
             Row {
                 NewTransferFab(newTransferFabType = NewTransferFabType.BOTTOM_BAR)
-                Spacer(modifier = Modifier.width(Margin.Large))
+                Spacer(Modifier.width(Margin.Large))
                 NewTransferFab(newTransferFabType = NewTransferFabType.EMPTY_STATE)
-                Spacer(modifier = Modifier.width(Margin.Large))
+                Spacer(Modifier.width(Margin.Large))
                 NewTransferFab(newTransferFabType = NewTransferFabType.NAVIGATION_RAIL)
             }
         }

diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/components/SmallFileItem.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/components/SmallFileItem.kt
@@ -82,7 +82,7 @@ private fun SmallFileItemPreview(@PreviewParameter(FileUiListPreviewParameter::c
                     onRemove = {}
                 )
 
-                Spacer(modifier = Modifier.height(Margin.Medium))
+                Spacer(Modifier.height(Margin.Medium))
 
                 SmallFileItem(
                     file = file,

diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/components/SwissTransferBottomSheet.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/components/SwissTransferBottomSheet.kt
@@ -103,7 +103,7 @@ private fun BottomSheetContent(
 
         imageVector?.let {
             Image(modifier = paddedModifier, imageVector = imageVector, contentDescription = null)
-            Spacer(modifier = Modifier.height(Margin.Large))
+            Spacer(Modifier.height(Margin.Large))
         }
 
         title?.let {
@@ -114,7 +114,7 @@ private fun BottomSheetContent(
                 color = SwissTransferTheme.colors.primaryTextColor,
                 textAlign = TextAlign.Center,
             )
-            Spacer(modifier = Modifier.height(Margin.Large))
+            Spacer(Modifier.height(Margin.Large))
         }
 
         description?.let {
@@ -125,12 +125,12 @@ private fun BottomSheetContent(
                 color = SwissTransferTheme.colors.secondaryTextColor,
                 textAlign = TextAlign.Center,
             )
-            Spacer(modifier = Modifier.height(Margin.Large))
+            Spacer(Modifier.height(Margin.Large))
         }
 
         content?.let {
             it()
-            Spacer(modifier = Modifier.height(Margin.Large))
+            Spacer(Modifier.height(Margin.Large))
         }
 
         DoubleButtonCombo(topButton = topButton, bottomButton = bottomButton)

diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/components/SwissTransferFab.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/components/SwissTransferFab.kt
@@ -73,7 +73,7 @@ private fun SwissTransferFabPreview() {
     SwissTransferTheme {
         Row {
             SwissTransferFab(onClick = {})
-            Spacer(modifier = Modifier.width(Margin.Large))
+            Spacer(Modifier.width(Margin.Large))
             SwissTransferFab(fabType = FabType.BIG, onClick = {})
         }
     }

diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/components/transfer/TransferItem.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/components/transfer/TransferItem.kt
@@ -87,14 +87,14 @@ fun TransferItem(
                     overflow = TextOverflow.MiddleEllipsis,
                 )
 
-                Spacer(modifier = Modifier.height(Margin.Mini))
+                Spacer(Modifier.height(Margin.Mini))
                 TextDotText(
                     firstText = { uploadedSize },
                     secondText = { expiryText },
                     optionalSecondTextColor = expiryColor,
                 )
 
-                Spacer(modifier = Modifier.height(Margin.Mini))
+                Spacer(Modifier.height(Margin.Mini))
                 ContextualFlowRow(
                     itemCount = files.count(),
                     maxLines = 1,
@@ -105,7 +105,7 @@ fun TransferItem(
                 }
             }
 
-            Spacer(modifier = Modifier.width(Margin.Medium))
+            Spacer(Modifier.width(Margin.Medium))
             Icon(
                 imageVector = AppIcons.ChevronRightThick,
                 contentDescription = null,

diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/screen/main/sent/SentEmptyScreen.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/screen/main/sent/SentEmptyScreen.kt
@@ -51,14 +51,14 @@ fun SentEmptyScreen() {
             style = SwissTransferTheme.typography.specificMedium32,
             textAlign = TextAlign.Center,
         )
-        Spacer(modifier = Modifier.height(Margin.Medium))
+        Spacer(Modifier.height(Margin.Medium))
         Text(
             modifier = Modifier.widthIn(max = maxWidth),
             text = stringResource(id = R.string.firstTransferDescription),
             style = SwissTransferTheme.typography.bodyRegular,
             color = SwissTransferTheme.colors.secondaryTextColor,
         )
-        Spacer(modifier = Modifier.height(Margin.Medium))
+        Spacer(Modifier.height(Margin.Medium))
         ConstraintLayout {
             val (icon, fab) = createRefs()
 

diff --git a/.../main/java/com/infomaniak/swisstransfer/ui/screen/main/settings/components/SettingItem.kt b/.../main/java/com/infomaniak/swisstransfer/ui/screen/main/settings/components/SettingItem.kt
@@ -94,7 +94,7 @@ private fun SettingItemContent(
             } else {
                 Image(imageVector = it, contentDescription = null)
             }
-            Spacer(modifier = Modifier.width(Margin.Medium))
+            Spacer(Modifier.width(Margin.Medium))
         }
 
         Column(modifier = Modifier.weight(1.0f)) {

diff --git a/...va/com/infomaniak/swisstransfer/ui/screen/main/settings/components/SingleSelectOptions.kt b/...va/com/infomaniak/swisstransfer/ui/screen/main/settings/components/SingleSelectOptions.kt
@@ -67,15 +67,15 @@ private fun SettingOptionItem(item: SettingOption, isSelected: Boolean, onClick:
         ) {
             item.icon?.let {
                 Image(imageVector = it, contentDescription = null)
-                Spacer(modifier = Modifier.width(Margin.Medium))
+                Spacer(Modifier.width(Margin.Medium))
             } ?: run {
                 // Make sure the items with no icons have the same height as the ones with icons
-                Spacer(modifier = Modifier.height(Margin.Large))
+                Spacer(Modifier.height(Margin.Large))
             }
 
             Text(text = item.title(), Modifier.weight(1.0f), style = SwissTransferTheme.typography.bodyRegular)
 
-            if (isSelected) Spacer(modifier = Modifier.width(Margin.Medium))
+            if (isSelected) Spacer(Modifier.width(Margin.Medium))
             AnimatedVisibility(
                 visible = isSelected,
                 enter = scaleIn(),

diff --git a/...java/com/infomaniak/swisstransfer/ui/screen/main/transferdetails/TransferDetailsScreen.kt b/...java/com/infomaniak/swisstransfer/ui/screen/main/transferdetails/TransferDetailsScreen.kt
@@ -177,13 +177,13 @@ private fun ColumnScope.FilesList(
         setUidCheckStatus = { fileUid, isChecked -> setFileCheckStatus(fileUid, isChecked) },
         header = {
             Column {
-                Spacer(modifier = Modifier.height(Margin.Large))
+                Spacer(Modifier.height(Margin.Large))
                 TransferInfo(transfer)
-                Spacer(modifier = Modifier.height(Margin.Large))
+                Spacer(Modifier.height(Margin.Large))
                 if (shouldDisplayRecipients) TransferRecipients(transferRecipients)
-                if (shouldDisplayRecipients && shouldDisplayMessage) Spacer(modifier = Modifier.height(Margin.Mini))
+                if (shouldDisplayRecipients && shouldDisplayMessage) Spacer(Modifier.height(Margin.Mini))
                 if (shouldDisplayMessage) TransferMessage(transfer.message!!)
-                if (shouldDisplayRecipients || shouldDisplayMessage) Spacer(modifier = Modifier.height(Margin.Large))
+                if (shouldDisplayRecipients || shouldDisplayMessage) Spacer(Modifier.height(Margin.Large))
                 TransferContentHeader()
             }
         },
@@ -197,7 +197,7 @@ private fun TransferRecipients(recipients: List<String>) {
         style = SwissTransferTheme.typography.bodySmallRegular,
         color = SwissTransferTheme.colors.secondaryTextColor,
     )
-    Spacer(modifier = Modifier.height(Margin.Mini))
+    Spacer(Modifier.height(Margin.Mini))
     EmailsFlowRow(recipients)
 }
 
@@ -208,7 +208,7 @@ private fun TransferMessage(transferMessage: String) {
         style = SwissTransferTheme.typography.bodySmallRegular,
         color = SwissTransferTheme.colors.secondaryTextColor,
     )
-    Spacer(modifier = Modifier.height(Margin.Medium))
+    Spacer(Modifier.height(Margin.Medium))
     SwissTransferCard {
         Text(
             text = transferMessage,
@@ -246,14 +246,14 @@ private fun BottomBar(direction: TransferDirection, isMultiselectOn: () -> Boole
 
                 when (direction) {
                     TransferDirection.SENT -> {
-                        Spacer(modifier = Modifier.width(Margin.Medium))
+                        Spacer(Modifier.width(Margin.Medium))
                         BottomBarButton(BottomBarItem.QR_CODE, onClick)
 
-                        Spacer(modifier = Modifier.width(Margin.Medium))
+                        Spacer(Modifier.width(Margin.Medium))
                         BottomBarButton(BottomBarItem.PASSWORD, onClick)
                     }
                     TransferDirection.RECEIVED -> {
-                        Spacer(modifier = Modifier.width(Margin.Medium))
+                        Spacer(Modifier.width(Margin.Medium))
                         BottomBarButton(BottomBarItem.DOWNLOAD, onClick)
                     }
                 }
@@ -275,7 +275,7 @@ private fun RowScope.BottomBarButton(item: BottomBarItem, onClick: (BottomBarIte
             horizontalAlignment = Alignment.CenterHorizontally,
         ) {
             Icon(item.icon, null)
-            Spacer(modifier = Modifier.height(Margin.Micro))
+            Spacer(Modifier.height(Margin.Micro))
             Text(stringResource(item.label))
         }
     }

diff --git a/...m/infomaniak/swisstransfer/ui/screen/main/transferdetails/components/QrCodeBottomSheet.kt b/...m/infomaniak/swisstransfer/ui/screen/main/transferdetails/components/QrCodeBottomSheet.kt
@@ -49,9 +49,9 @@ fun QrCodeBottomSheet(isVisible: () -> Boolean, transferUrl: String, closeBottom
         descriptionRes = R.string.shareQrCodeDescription,
     ) {
         Column {
-            Spacer(modifier = Modifier.height(Margin.Large))
+            Spacer(Modifier.height(Margin.Large))
             QrCode(transferUrl)
-            Spacer(modifier = Modifier.height(Margin.Large))
+            Spacer(Modifier.height(Margin.Large))
         }
     }
 }

diff --git a/...nfomaniak/swisstransfer/ui/screen/newtransfer/importfiles/components/ImportedFilesCard.kt b/...nfomaniak/swisstransfer/ui/screen/newtransfer/importfiles/components/ImportedFilesCard.kt
@@ -61,7 +61,7 @@ fun ImportedFilesCard(
                 secondText = { formatSpaceLeft(humanReadableSize) },
                 modifier = Modifier.padding(start = Margin.Medium),
             )
-            Spacer(modifier = Modifier.weight(1f))
+            Spacer(Modifier.weight(1.0f))
             Icon(
                 imageVector = AppIcons.ChevronRightThick,
                 contentDescription = null,

diff --git a/...fomaniak/swisstransfer/ui/screen/newtransfer/importfiles/components/TransferOptionType.kt b/...fomaniak/swisstransfer/ui/screen/newtransfer/importfiles/components/TransferOptionType.kt
@@ -50,15 +50,15 @@ fun TransferOptionType(
             tint = SwissTransferTheme.materialColors.primary,
             contentDescription = null,
         )
-        Spacer(modifier = Modifier.width(Margin.Small))
+        Spacer(Modifier.width(Margin.Small))
         Text(
             text = stringResource(transferOptionType.buttonText),
             color = SwissTransferTheme.materialColors.primary,
             style = SwissTransferTheme.typography.bodySmallMedium,
         )
-        Spacer(modifier = Modifier.weight(1.0f))
+        Spacer(Modifier.weight(1.0f))
         SettingValue(selectedSetting)
-        Spacer(modifier = Modifier.width(Margin.Small))
+        Spacer(Modifier.width(Margin.Small))
         Icon(
             modifier = Modifier.size(Dimens.SmallIconSize),
             imageVector = AppIcons.ChevronRightThick,

diff --git a/...fomaniak/swisstransfer/ui/screen/newtransfer/importfiles/components/TransferTypeButton.kt b/...fomaniak/swisstransfer/ui/screen/newtransfer/importfiles/components/TransferTypeButton.kt
@@ -52,7 +52,7 @@ fun TransferTypeButton(
         contentPadding = PaddingValues(Margin.Medium),
     ) {
         Icon(modifier = Modifier.size(Dimens.IconSize), imageVector = transferType.buttonIcon, contentDescription = null)
-        Spacer(modifier = Modifier.width(Margin.Mini))
+        Spacer(Modifier.width(Margin.Mini))
         Text(text = stringResource(transferType.buttonTextRes), style = SwissTransferTheme.typography.bodySmallRegular)
     }
 }

diff --git a/...in/java/com/infomaniak/swisstransfer/ui/screen/newtransfer/upload/UploadProgressScreen.kt b/...in/java/com/infomaniak/swisstransfer/ui/screen/newtransfer/upload/UploadProgressScreen.kt
@@ -114,10 +114,10 @@ private fun UploadProgressScreen(
         ) {
             AdHeader(adScreenType)
 
-            Spacer(modifier = Modifier.height(Margin.Medium))
+            Spacer(Modifier.height(Margin.Medium))
             Text(stringResource(R.string.uploadProgressIndication))
             Progress(progressState, totalSizeInBytes)
-            Spacer(modifier = Modifier.height(Margin.Huge))
+            Spacer(Modifier.height(Margin.Huge))
         }
 
         if (showBottomSheet.get()) CancelUploadBottomSheet(onCancel = onCancel, closeButtonSheet = { showBottomSheet.set(false) })

diff --git a/...ava/com/infomaniak/swisstransfer/ui/screen/newtransfer/upload/UploadSuccessEmailScreen.kt b/...ava/com/infomaniak/swisstransfer/ui/screen/newtransfer/upload/UploadSuccessEmailScreen.kt
@@ -66,7 +66,7 @@ fun UploadSuccessEmailScreen(
                 description = pluralStringResource(TransferTypeUi.MAIL.descriptionRes!!, emails.count()),
             )
 
-            Spacer(modifier = Modifier.height(Margin.Medium))
+            Spacer(Modifier.height(Margin.Medium))
 
             EmailsFlowRow(
                 emails = emails,

diff --git a/...n/java/com/infomaniak/swisstransfer/ui/screen/newtransfer/upload/UploadSuccessQrScreen.kt b/...n/java/com/infomaniak/swisstransfer/ui/screen/newtransfer/upload/UploadSuccessQrScreen.kt
@@ -80,7 +80,7 @@ private fun ColumnScope.SuccessMessage(transferType: TransferTypeUi, transferUrl
         horizontalAlignment = Alignment.CenterHorizontally,
         verticalArrangement = Arrangement.Center,
     ) {
-        Spacer(modifier = Modifier.height(Margin.Medium))
+        Spacer(Modifier.height(Margin.Medium))
 
         Image(imageVector = AppIllus.Beers.image(), contentDescription = null)
 

diff --git a/...ain/java/com/infomaniak/swisstransfer/ui/screen/newtransfer/upload/components/AdHeader.kt b/...ain/java/com/infomaniak/swisstransfer/ui/screen/newtransfer/upload/components/AdHeader.kt
@@ -44,15 +44,15 @@ fun ColumnScope.AdHeader(adScreenType: UploadProgressAdType) {
             .verticalScroll(rememberScrollState()),
         horizontalAlignment = Alignment.CenterHorizontally,
     ) {
-        Spacer(modifier = Modifier.height(Margin.Giant))
+        Spacer(Modifier.height(Margin.Giant))
 
         HighlightedText(
             templateRes = R.string.uploadProgressTitleTemplate,
             argumentRes = R.string.uploadProgressTitleArgument,
             style = SwissTransferTheme.typography.bodyMedium,
         )
 
-        Spacer(modifier = Modifier.height(Margin.Huge))
+        Spacer(Modifier.height(Margin.Huge))
         Text(
             text = adScreenType.description(),
             modifier = Modifier.widthIn(max = Dimens.DescriptionWidth),