Revert "Calculate Bottom Commanding header height instead of using co…

…nstant (microsoft#2041)" This reverts commit d298dac.
joannaquu · Jul 17, 2024 · bc9d475 · bc9d475
1 parent 7d00d6a
commit bc9d475
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 11 deletions.
diff --git a/ios/FluentUI.Demo/FluentUI.Demo/Demos/BottomCommandingDemoController.swift b/ios/FluentUI.Demo/FluentUI.Demo/Demos/BottomCommandingDemoController.swift
@@ -46,8 +46,7 @@ class BottomCommandingDemoController: DemoController {
 
     private lazy var heroItems: [CommandingItem] = {
         return Array(1...25).map {
-            let title = ($0 == 4) ? "Two line item" : "Item"
-            let item = CommandingItem(title: title + String($0), image: homeImage, action: commandAction)
+            let item = CommandingItem(title: "Item " + String($0), image: homeImage, action: commandAction)
             item.selectedImage = homeSelectedImage
             item.isOn = ($0 % 3 == 1)
             item.isEnabled = ($0 % 2 == 1)

diff --git a/ios/FluentUI/Bottom Commanding/BottomCommandingController.swift b/ios/FluentUI/Bottom Commanding/BottomCommandingController.swift
@@ -402,6 +402,7 @@ open class BottomCommandingController: UIViewController, TokenizedControlInterna
         headerView.addSubview(heroCommandStack)
 
         let sheetController = BottomSheetController(headerContentView: headerView, expandedContentView: makeSheetExpandedContent(with: tableView))
+        sheetController.headerContentHeight = Constants.BottomSheet.headerHeight
         sheetController.hostedScrollView = tableView
         sheetController.isHidden = isHidden
         sheetController.shouldAlwaysFillWidth = sheetShouldAlwaysFillWidth
@@ -533,7 +534,6 @@ open class BottomCommandingController: UIViewController, TokenizedControlInterna
         } else {
             tableView.tableHeaderView = nil
         }
-        calculateHeaderHeight()
     }
 
     private func reloadHeroCommandOverflowStack() {
@@ -563,13 +563,6 @@ open class BottomCommandingController: UIViewController, TokenizedControlInterna
         }
     }
 
-    @discardableResult
-    private func calculateHeaderHeight() -> CGFloat {
-        let headerHeight = heroCommandStack.systemLayoutSizeFitting(UIView.layoutFittingCompressedSize).height + BottomSheetController.resizingHandleHeight
-        bottomSheetController?.headerContentHeight = headerHeight
-        return headerHeight
-    }
-
     private func updateAppearance() {
         guard isViewLoaded else {
             return
@@ -930,7 +923,7 @@ open class BottomCommandingController: UIViewController, TokenizedControlInterna
         let headerHeightWithoutBottomWhitespace = BottomCommandingTokenSet.handleHeaderHeight + maxHeroItemHeight
 
         // How much more whitespace is required at the bottom of the sheet header
-        let requiredBottomWhitespace = max(0, calculateHeaderHeight() - headerHeightWithoutBottomWhitespace)
+        let requiredBottomWhitespace = max(0, Constants.BottomSheet.headerHeight - headerHeightWithoutBottomWhitespace)
 
         // The safe area inset can fulfill some or all of our bottom whitespace requirement.
         // This is how much more we need, taking the inset into account.
@@ -1125,6 +1118,10 @@ open class BottomCommandingController: UIViewController, TokenizedControlInterna
             static let moreButtonIcon: UIImage? = UIImage.staticImageNamed("more-24x24")
             static let moreButtonTitle: String = "CommandingBottomBar.More".localized
         }
+
+        struct BottomSheet {
+            static let headerHeight: CGFloat = 66
+        }
     }
 }