fix: gap shorthands should be lowercase (#218)

Shopify · Jan 30, 2023 · 463426a · 463426a
1 parent 11a9812
commit 463426a
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/README.md b/README.md
@@ -372,7 +372,7 @@ The Restyle library comes with a number of predefined Restyle functions for your
 | color            | color                                                                                                                                                                                                                                                                                                                                  | colors      |
 | opacity          | opacity                                                                                                                                                                                                                                                                                                                                | _none_      |
 | visible          | display (maps `true` / `false` to `flex` / `none`)                                                                                                                                                                                                                                                                                     | _none_      |
-| spacing          | margin [m], marginTop [mt], marginRight [mr], marginBottom [mb], marginLeft [ml], marginStart [ms], marginEnd[me], marginHorizontal [mx], marginVertical [my], padding [p], paddingTop [pt], paddingRight [pr], paddingBottom [pb], paddingLeft [pl], paddingStart [ps], paddingEnd [pe], paddingHorizontal [px], paddingVertical [py], gap [g], rowGap [rG], columnGap [cG] | spacing     |
+| spacing          | margin [m], marginTop [mt], marginRight [mr], marginBottom [mb], marginLeft [ml], marginStart [ms], marginEnd[me], marginHorizontal [mx], marginVertical [my], padding [p], paddingTop [pt], paddingRight [pr], paddingBottom [pb], paddingLeft [pl], paddingStart [ps], paddingEnd [pe], paddingHorizontal [px], paddingVertical [py], gap [g], rowGap [rg], columnGap [cg] | spacing     |
 | layout           | width, height, minWidth, maxWidth, minHeight, maxHeight, overflow, aspectRatio, alignContent, alignItems, alignSelf, justifyContent, flex, flexBasis, flexDirection, flexGrow, flexShrink, flexWrap                                                                                                                                    | _none_      |
 | position         | position, top, right, bottom, left, start, end                                                                                                                                                                                                                                                                                         | _none_      |
 | position         | zIndex                                                                                                                                                                                                                                                                                                                                 | zIndices    |

diff --git a/src/restyleFunctions.ts b/src/restyleFunctions.ts
@@ -48,8 +48,8 @@ const spacingPropertiesShorthand = {
   ps: 'paddingStart',
   pe: 'paddingEnd',
   g: 'gap',
-  rG: 'rowGap',
-  cG: 'columnGap',
+  rg: 'rowGap',
+  cg: 'columnGap',
 };
 
 const typographyProperties = {

diff --git a/src/test/createRestyleComponent.test.tsx b/src/test/createRestyleComponent.test.tsx
@@ -211,7 +211,7 @@ describe('createRestyleComponent', () => {
     it('passes gap shorthands as gap values', () => {
       const {root} = render(
         <ThemeProvider theme={theme}>
-          <Component g="s" cG="s" rG="s" />
+          <Component g="s" cg="s" rg="s" />
         </ThemeProvider>,
       );
       expect(root.findByType(View).props).toStrictEqual({