New method for separator hiding (#2068) (#2073)

(cherry picked from commit 30b8885) (cherry picked from commit 60aea2b) Co-authored-by: Lukas Capkovic <3610850+lcapkovic@users.noreply.github.com>
microsoft · Jul 18, 2024 · cda11a8 · cda11a8
1 parent 04f2322
commit cda11a8
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/ios/FluentUI/Other Cells/ActionsCell.swift b/ios/FluentUI/Other Cells/ActionsCell.swift
@@ -187,9 +187,9 @@ open class ActionsCell: UITableViewCell, TokenizedControlInternal {
         // A hacky way to hide the system separator by squeezing it just enough to have zero width.
         // This is the only known way to hide the separator without making the UITableView do it for us.
         let boundsWidth = bounds.width
-        if separatorInset.left != boundsWidth || separatorInset.right != 0 {
-            separatorInset.left = boundsWidth
-            separatorInset.right = 0
+        let targetSystemSeparatorInset = UIEdgeInsets(top: 0, left: boundsWidth, bottom: 0, right: 0)
+        if separatorInset.left < boundsWidth {
+            separatorInset = targetSystemSeparatorInset
         }
 
         layoutHorizontalSeparator(topSeparator, with: topSeparatorType, at: 0)

diff --git a/ios/FluentUI/Table View/TableViewCell.swift b/ios/FluentUI/Table View/TableViewCell.swift
@@ -1517,9 +1517,9 @@ open class TableViewCell: UITableViewCell, TokenizedControlInternal {
         // A hacky way to hide the system separator by squeezing it just enough to have zero width.
         // This is the only known way to hide the separator without making the UITableView do it for us.
         let boundsWidth = bounds.width
-        if separatorInset.left != boundsWidth || separatorInset.right != 0 {
-            separatorInset.left = boundsWidth
-            separatorInset.right = 0
+        let targetSystemSeparatorInset = UIEdgeInsets(top: 0, left: boundsWidth, bottom: 0, right: 0)
+        if separatorInset.left < boundsWidth {
+            separatorInset = targetSystemSeparatorInset
         }
 
         layoutSeparator(topSeparator, with: topSeparatorType, at: 0)