Skip to content

Commit

Permalink
fix: code review
Browse files Browse the repository at this point in the history
  • Loading branch information
danielsimao committed Mar 13, 2024
1 parent 20fba4b commit b1c3f72
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 8 deletions.
7 changes: 3 additions & 4 deletions packages/components/src/Flex/Flex.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,9 @@ import { Flex, FlexProps } from '.';
export default {
title: 'Layout/Flex',
component: Flex,

// args: {
// gap: 'md'
// },
args: {
gap: 'md'
},
render: (args: FlexProps) => (
<Flex {...args}>
<div style={{ height: 60, width: 100, background: 'red' }} />
Expand Down
8 changes: 4 additions & 4 deletions packages/components/src/utils/responsive.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { ResponsiveProp, Spacing, Theme } from '@interlay/theme';
import { css } from 'styled-components';
import { ResponsiveProp, Spacing } from '@interlay/theme';
import { DefaultTheme, css } from 'styled-components';

const getSpacingResponsiveCSS = (theme: Theme, attribute: string, prop?: ResponsiveProp<Spacing>) =>
const getSpacingResponsiveCSS = (theme: DefaultTheme, attribute: string, prop?: ResponsiveProp<Spacing>) =>
typeof prop === 'object'
? css`
${prop.xs && theme.breakpoints.media.xs`${attribute}: ${theme.spacing(prop.xs)};`}
Expand All @@ -13,7 +13,7 @@ const getSpacingResponsiveCSS = (theme: Theme, attribute: string, prop?: Respons
: prop && `${attribute}:${theme.spacing(prop)};`;

const getResponsiveCSS = (
theme: Theme,
theme: DefaultTheme,
attribute: string,
prop?: ResponsiveProp<string | number | boolean>,
condition?: (prop: string | number | boolean) => string | number
Expand Down

0 comments on commit b1c3f72

Please sign in to comment.