Skip to content

Commit 5f47953

Browse files
gwyneplainejesstelford
authored andcommitted
Docs fix Box implementation to use more specific padding values first before defaulting to the overarching padding value
1 parent 4d4bf8c commit 5f47953

File tree

1 file changed

+28
-4
lines changed
  • polaris.shopify.com/src/components/Box

1 file changed

+28
-4
lines changed

polaris.shopify.com/src/components/Box/index.tsx

Lines changed: 28 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,15 +41,39 @@ export const Box = forwardRef(
4141
minWidth,
4242
maxWidth,
4343
padding = '0',
44+
paddingBlockStart,
45+
paddingBlockEnd,
46+
paddingInlineStart,
47+
paddingInlineEnd,
4448
...props
4549
},
4650
forwardedRef,
4751
) => {
4852
const style = {
49-
...getResponsiveProps('box', 'padding-block-start', 'space', padding),
50-
...getResponsiveProps('box', 'padding-block-end', 'space', padding),
51-
...getResponsiveProps('box', 'padding-inline-start', 'space', padding),
52-
...getResponsiveProps('box', 'padding-inline-end', 'space', padding),
53+
...getResponsiveProps(
54+
'box',
55+
'padding-block-start',
56+
'space',
57+
paddingBlockStart || padding,
58+
),
59+
...getResponsiveProps(
60+
'box',
61+
'padding-block-end',
62+
'space',
63+
paddingBlockEnd || padding,
64+
),
65+
...getResponsiveProps(
66+
'box',
67+
'padding-inline-start',
68+
'space',
69+
paddingInlineStart || padding,
70+
),
71+
...getResponsiveProps(
72+
'box',
73+
'padding-inline-end',
74+
'space',
75+
paddingInlineEnd || padding,
76+
),
5377
'--pc-box-min-height': minHeight,
5478
'--px-box-min-width': minWidth,
5579
'--px-box-max-width': maxWidth,

0 commit comments

Comments
 (0)