Fix horizontal padding issue for sidebar for macos

sentryco · Oct 26, 2024 · 0734a4a · 0734a4a
1 parent 39378c4
commit 0734a4a
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 1 deletion.
diff --git a/ExampleProject/ExampleProject/example/style/ListButtonStyle.swift b/ExampleProject/ExampleProject/example/style/ListButtonStyle.swift
@@ -22,7 +22,7 @@ struct ListButtonStyle: ButtonStyle {
       HStack(spacing: 0) {
          configuration.label // text instance
             .foregroundStyle(textColor)
-            .padding(.horizontal)
+            // .padding(.horizontal)
             .font(.system(size: 24, weight: .regular))
          Spacer()
       }

diff --git a/ExampleProject/ExampleProject/example/view/main/MainList/MainList+Content.swift b/ExampleProject/ExampleProject/example/view/main/MainList/MainList+Content.swift
@@ -13,6 +13,7 @@ extension MainList {
          rowViews
       }
       .padding(.vertical, 12) // Add top and bottom padding for the list etc
+      .padding(.horizontal) // Adds horizontal padding
    }
    /**
     * List content

diff --git a/ExampleProject/ExampleProject/example/view/sidebar/SideBarList/SideBarList+Content.swift b/ExampleProject/ExampleProject/example/view/sidebar/SideBarList/SideBarList+Content.swift
@@ -17,8 +17,10 @@ extension SideBarList {
       }
       .scrollContentBackground(.hidden) // remove list bg
       .environment(\.defaultMinListRowHeight, .zero) // ⚠️️ key to resetting topSpacer in the Listcontainer
+      #if os(iOS) // these seem to have no effect for macOS
       .contentMargins(.vertical, .zero) // ⚠️️ key to removeing top margin, we cant use .contentMargins(.horizontal, .zero) as it will remove the rounded backgrounds for the section
       .contentMargins(.horizontal, .zero) // Sets the horizontal content margins to zero, removing any default horizontal spacing.
+      #endif
       .padding(.vertical, 12) // Adds vertical padding of 12 points to the list, enhancing visual separation between elements.
    }
    /**