Skip to content

Commit e96f79e

Browse files
author
Reed Es
committed
Better spacing defaults for Grid
1 parent 7c87474 commit e96f79e

2 files changed

Lines changed: 12 additions & 12 deletions

File tree

README.md

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -416,9 +416,9 @@ Stack configuration is optional.
416416

417417
`TablerStackConfig<Element>.init` parameters:
418418

419-
- `rowPadding: EdgeInsets` - Stack-specific default, varies by platform
420-
- `headerSpacing: CGFloat` - Stack-specific default, varies by platform
421-
- `rowSpacing: CGFloat` - Stack-specific default, varies by platform
419+
- `rowPadding: EdgeInsets` - Stack-specific default; varies by platform
420+
- `headerSpacing: CGFloat` - Stack-specific default; varies by platform
421+
- `rowSpacing: CGFloat` - Stack-specific default of 0
422422
- `filter: Filter?` - with a default of `nil`, indicating no filtering
423423
- `onHover: (Element.ID, Bool) -> Void` - defaults to `{ _,_ in }`
424424
- `tablePadding: EdgeInsets` - per Stack defaults
@@ -434,12 +434,12 @@ Grid configuration is required, where you supply a `GridItem` array.
434434

435435
- `gridItems: [GridItem]` - required
436436
- `alignment: HorizontalAlignment` - `LazyVGrid` alignment, with a default of `.leading`
437-
- `itemPadding: EdgeInsets` - with a default of `.init()`, indicating no padding
438-
- `headerSpacing: CGFloat` - Grid-specific default, varies by platform
439-
- `rowSpacing: CGFloat` - Grid-specific default, varies by platform
437+
- `itemPadding: EdgeInsets` - Grid-specific defaults, varies by platform
438+
- `headerSpacing: CGFloat` - Grid-specific default; varies by platform
439+
- `rowSpacing: CGFloat` - Grid-specific default of 0
440440
- `filter: Filter?` - with a default of `nil`, indicating no filtering
441441
- `onHover: (Element.ID, Bool) -> Void` - defaults to `{ _,_ in }`
442-
- `tablePadding: EdgeInsets` - Grid-specific default, varies by platform
442+
- `tablePadding: EdgeInsets` - Grid-specific default; varies by platform
443443
- `sortIndicatorForward: AnyView` - per Base defaults
444444
- `sortIndicatorReverse: AnyView` - per Base defaults
445445
- `sortIndicatorNeutral: AnyView` - per Base defaults

Sources/Grid/TablerGridConfig.swift

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,16 +20,16 @@ import SwiftUI
2020

2121
public enum TablerGridConfigDefaults {
2222
#if os(macOS)
23-
public static let headerSpacing: CGFloat = 8
24-
public static let rowSpacing: CGFloat = 8
23+
public static let headerSpacing: CGFloat = 4
24+
public static let itemPadding = EdgeInsets(top: 4, leading: 0, bottom: 4, trailing: 0)
2525
public static let tablePadding = EdgeInsets(top: 14, leading: 16, bottom: 15, trailing: 16)
2626
#elseif os(iOS)
27-
public static let headerSpacing: CGFloat = 23.5
28-
public static let rowSpacing: CGFloat = 23.5
27+
public static let headerSpacing: CGFloat = 12
28+
public static let itemPadding = EdgeInsets(top: 11.5, leading: 0, bottom: 12, trailing: 0)
2929
public static let tablePadding = EdgeInsets(top: 46, leading: 32, bottom: 20, trailing: 32)
3030
#endif
3131

32-
public static let itemPadding: EdgeInsets = .init()
32+
public static let rowSpacing: CGFloat = 0
3333
public static let alignment: HorizontalAlignment = .leading
3434
}
3535

0 commit comments

Comments
 (0)