From c8e42d038547e1c1a07f0452a3caafd505dc76de Mon Sep 17 00:00:00 2001
From: Ayaan Shakil <98223682+ayanshakil@users.noreply.github.com>
Date: Fri, 14 Jun 2024 11:37:38 +0530
Subject: [PATCH] bottom sheet width set to 60% in landscape mode and aligned
 center (#659)

* bottom sheet width set to 60% in landscape mode and aligned center

* changing default max landscape width to 1F

---------

Co-authored-by: Ayaan Shakil <t-ashakil@microsoft.com>
---
 .../theme/token/controlTokens/BottomSheetTokens.kt     |  3 +++
 .../fluentui/tokenized/bottomsheet/BottomSheet.kt      | 10 +++++++++-
 2 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/fluentui_core/src/main/java/com/microsoft/fluentui/theme/token/controlTokens/BottomSheetTokens.kt b/fluentui_core/src/main/java/com/microsoft/fluentui/theme/token/controlTokens/BottomSheetTokens.kt
index a3f4a1286..72d81385f 100644
--- a/fluentui_core/src/main/java/com/microsoft/fluentui/theme/token/controlTokens/BottomSheetTokens.kt
+++ b/fluentui_core/src/main/java/com/microsoft/fluentui/theme/token/controlTokens/BottomSheetTokens.kt
@@ -46,4 +46,7 @@ open class BottomSheetTokens : IControlToken, Parcelable {
 
     @Composable
     open fun scrimOpacity(bottomSheetInfo: BottomSheetInfo): Float = 0.32F
+
+    @Composable
+    open fun maxLandscapeWidth (bottomSheetInfo: BottomSheetInfo): Float = 1F
 }
\ No newline at end of file
diff --git a/fluentui_drawer/src/main/java/com/microsoft/fluentui/tokenized/bottomsheet/BottomSheet.kt b/fluentui_drawer/src/main/java/com/microsoft/fluentui/tokenized/bottomsheet/BottomSheet.kt
index 9e3393c23..aadfd5de5 100644
--- a/fluentui_drawer/src/main/java/com/microsoft/fluentui/tokenized/bottomsheet/BottomSheet.kt
+++ b/fluentui_drawer/src/main/java/com/microsoft/fluentui/tokenized/bottomsheet/BottomSheet.kt
@@ -5,6 +5,7 @@
 
 package com.microsoft.fluentui.tokenized.bottomsheet
 
+import android.content.res.Configuration
 import android.view.*
 import androidx.compose.animation.core.AnimationSpec
 import androidx.compose.foundation.*
@@ -29,6 +30,7 @@ import androidx.compose.ui.graphics.Shape
 import androidx.compose.ui.input.nestedscroll.nestedScroll
 import androidx.compose.ui.input.pointer.pointerInput
 import androidx.compose.ui.layout.*
+import androidx.compose.ui.platform.LocalConfiguration
 import androidx.compose.ui.platform.LocalContext
 import androidx.compose.ui.platform.testTag
 import androidx.compose.ui.res.painterResource
@@ -251,6 +253,7 @@ fun BottomSheet(
         tokens.scrimColor(bottomSheetInfo).copy(alpha = scrimOpacity)
 
     val scope = rememberCoroutineScope()
+    val maxLandscapeWidth :Float= tokens.maxLandscapeWidth(bottomSheetInfo)
 
     BoxWithConstraints(modifier) {
         val fullHeight = constraints.maxHeight.toFloat()
@@ -260,6 +263,7 @@ fun BottomSheet(
         Box(
             Modifier
                 .fillMaxSize()
+                .align(Alignment.Center)
                 .semantics {
                     if (!sheetState.isVisible) {
                         expand {
@@ -301,10 +305,14 @@ fun BottomSheet(
                 )
             }
         }
+        val configuration = LocalConfiguration.current
 
         Box(
             Modifier
-                .fillMaxWidth()
+                .fillMaxWidth(
+                    if(configuration.orientation == Configuration.ORIENTATION_LANDSCAPE)maxLandscapeWidth
+                    else 1F
+                )
                 .nestedScroll(
                     if (!enableSwipeDismiss && sheetState.offset.value >= (fullHeight - dpToPx(
                             peekHeight