diff --git a/app/screenshots/gplay/debug/com.owncloud.android.ui.dialog.DialogFragmentIT_testFileActionsBottomSheet.png b/app/screenshots/gplay/debug/com.owncloud.android.ui.dialog.DialogFragmentIT_testFileActionsBottomSheet.png index c6c31f47304e..ab19cd1bc591 100644 Binary files a/app/screenshots/gplay/debug/com.owncloud.android.ui.dialog.DialogFragmentIT_testFileActionsBottomSheet.png and b/app/screenshots/gplay/debug/com.owncloud.android.ui.dialog.DialogFragmentIT_testFileActionsBottomSheet.png differ diff --git a/app/src/main/java/com/nextcloud/ui/fileactions/FileActionsBottomSheet.kt b/app/src/main/java/com/nextcloud/ui/fileactions/FileActionsBottomSheet.kt index de58c75a9d78..4ec9447f7ff2 100644 --- a/app/src/main/java/com/nextcloud/ui/fileactions/FileActionsBottomSheet.kt +++ b/app/src/main/java/com/nextcloud/ui/fileactions/FileActionsBottomSheet.kt @@ -79,13 +79,13 @@ class FileActionsBottomSheet : BottomSheetDialogFragment(), Injectable { @Inject lateinit var syncedFolderProvider: SyncedFolderProvider - lateinit var viewModel: FileActionsViewModel + private lateinit var viewModel: FileActionsViewModel private var _binding: FileActionsBottomSheetBinding? = null private val binding get() = _binding!! - lateinit var componentsGetter: ComponentsGetter + private lateinit var componentsGetter: ComponentsGetter private val thumbnailAsyncTasks = mutableListOf() @@ -109,6 +109,8 @@ class FileActionsBottomSheet : BottomSheetDialogFragment(), Injectable { bottomSheetDialog.behavior.state = BottomSheetBehavior.STATE_EXPANDED bottomSheetDialog.behavior.skipCollapsed = true + viewThemeUtils.platform.colorViewBackground(binding.bottomSheet, ColorRole.SURFACE) + return binding.root } @@ -125,11 +127,13 @@ class FileActionsBottomSheet : BottomSheetDialogFragment(), Injectable { displayActions(state.actions) displayTitle(state.titleFile) } + is FileActionsViewModel.UiState.LoadedForMultipleFiles -> { setMultipleFilesThumbnail() displayActions(state.actions) displayTitle(state.fileCount) } + FileActionsViewModel.UiState.Loading -> {} FileActionsViewModel.UiState.Error -> { context?.let { @@ -195,11 +199,11 @@ class FileActionsBottomSheet : BottomSheetDialogFragment(), Injectable { private fun toggleLoadingOrContent(state: FileActionsViewModel.UiState) { if (state is FileActionsViewModel.UiState.Loading) { binding.bottomSheetLoading.isVisible = true - binding.bottomSheetContent.isVisible = false + binding.bottomSheetHeader.isVisible = false viewThemeUtils.platform.colorCircularProgressBar(binding.bottomSheetLoading, ColorRole.PRIMARY) } else { binding.bottomSheetLoading.isVisible = false - binding.bottomSheetContent.isVisible = true + binding.bottomSheetHeader.isVisible = true } } diff --git a/app/src/main/res/layout/file_actions_bottom_sheet.xml b/app/src/main/res/layout/file_actions_bottom_sheet.xml index 4a0a2f283e09..9275cf5a511c 100644 --- a/app/src/main/res/layout/file_actions_bottom_sheet.xml +++ b/app/src/main/res/layout/file_actions_bottom_sheet.xml @@ -19,73 +19,75 @@ ~ License along with this program. If not, see . ~ --> - - - - + android:orientation="vertical"> - + - + android:layout_gravity="center" + android:layout_marginBottom="@dimen/standard_margin" + android:indeterminate="true" + tools:visibility="gone" /> + + - + layout="@layout/file_thumbnail" + android:layout_width="wrap_content" + android:layout_height="wrap_content" + android:layout_marginStart="12dp" /> - + - + - - - - - + android:layout_height="wrap_content"> + + + + + + + + diff --git a/app/src/main/res/values/dims.xml b/app/src/main/res/values/dims.xml index 6b3fe89a1e9c..2b389cefa22c 100644 --- a/app/src/main/res/values/dims.xml +++ b/app/src/main/res/values/dims.xml @@ -23,7 +23,6 @@ 40dp 56dp 80dp - 112dp 40dp 128dp 8dp