Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bugfix: fix bugs #113

Merged
merged 1 commit into from
May 7, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 27 additions & 16 deletions src/ui/atoms/list/usage.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -195,14 +195,19 @@ Size of list item is controlled by the `component-level` block property not from
</ListContainer>
</block.S>
<block.M>
<List
variant="primary"
list={[
{ left: <IconInfo />, id: 0, text: 'Red' },
{ left: <IconInfo />, id: 1, text: 'Green' },
{ left: <IconInfo />, id: 2, text: 'Blue' },
]}
/>
<ListContainer variant="primary" style={{ width: '100%' }}>
{links.map(({ id, content, disabled }) => (
<ListItem
key={id}
as="a"
href={`#${id}`}
disabled={disabled}
text={content}
iconRight={<IconInfo />}
iconLeft={<IconInfo />}
/>
))}
</ListContainer>
</block.M>
<block.L>
<ListContainer variant="primary" style={{ width: '100%' }}>
Expand Down Expand Up @@ -236,14 +241,20 @@ Size of list item is controlled by the `component-level` block property not from
</ListContainer>
</block.XL>
<block.H>
<List
variant="primary"
list={[
{ left: <IconInfo />, id: 0, text: 'Red' },
{ left: <IconInfo />, id: 1, text: 'Green' },
{ left: <IconInfo />, id: 2, text: 'Blue' },
]}
/>
<ListContainer variant="primary" style={{ width: '100%' }} as="ul">
{links.map(({ id, content, disabled }) => (
<ListItem
as="a"
disabled={disabled}
href={`#${id}`}
iconLeft={<IconInfo />}
iconRight={<IconInfo />}
key={id}
text={content}
variant="primary"
/>
))}
</ListContainer>
</block.H>
</Playground>

Expand Down
4 changes: 2 additions & 2 deletions src/ui/atoms/table/usage.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -87,8 +87,8 @@ To set number of columns pass `columns` prop to the Table component.
</Tr>
</Thead>
<Tbody>
{tableRows.map((row) => (
<Tr>
{tableRows.map((row, key) => (
<Tr key={key}>
{tableHead.map(({ id }) => (
<Td key={id}>{row[id]}</Td>
))}
Expand Down
11 changes: 9 additions & 2 deletions src/ui/molecules/checkbox/usage.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -71,8 +71,15 @@ Primary variant should be used to focus user attention. The disabled checkbox is
<State initial={false} change={(i) => !i}>
{(value, change) => (
<>
<Checkbox checked id="primary" text="Primary" variant="primary" />
<Checkbox checked disabled id="disabled" text="Disabled" variant="primary" />
<Checkbox checked onChange={change} id="primary" text="Primary" variant="primary" />
<Checkbox
checked
onChange={change}
disabled
id="disabled"
text="Disabled"
variant="primary"
/>
</>
)}
</State>
Expand Down
10 changes: 5 additions & 5 deletions src/ui/molecules/field/usage.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ or also as a label of group of component, eg checkbox or radiobutton groups.
onChange={() => console.info('On textarea change')}
variant="primary"
overflow="hidden"
isDisabled
disabled
/>
</Field>
<Field label="Label for input">
Expand All @@ -81,7 +81,7 @@ Labels and children component can be replaced in row
<State initial={false} change={(i) => !i}>
{(value, change) => (
<Field label="Label of item" row>
<Switch isChecked={value} onChange={change} id="roundTwo" variant="primary" />
<Switch checked={value} onChange={change} id="roundTwo" variant="primary" />
</Field>
)}
</State>
Expand All @@ -98,7 +98,7 @@ or in column variants
<State initial={false} change={(i) => !i}>
{(value, change) => (
<Field label="Label of item">
<Switch isChecked={value} onChange={change} id="roundOne" variant="primary" />
<Switch checked={value} onChange={change} id="roundOne" variant="primary" />
</Field>
)}
</State>
Expand All @@ -116,7 +116,7 @@ Error variant can be used to focus user attention on error.
onChange={() => console.info('On textarea change')}
variant="danger"
overflow="hidden"
isDisabled
disabled
/>
</Field>
<Field label="Input with error" variant="danger">
Expand All @@ -131,7 +131,7 @@ Error variant can be used to focus user attention on error.
<State initial={false} change={(i) => !i}>
{(value, change) => (
<Field label="Switch with error">
<Switch isChecked={value} onChange={change} id="roundOne" variant="danger" />
<Switch checked={value} onChange={change} id="roundOne" variant="danger" />
</Field>
)}
</State>
Expand Down
4 changes: 3 additions & 1 deletion src/ui/molecules/radio-button/usage.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,9 @@ Disabled radio button are uneditable. They have less opacity so that they appear

<Playground>
<State initial={false} change={(i) => !i}>
{(value, change) => <RadioButton id="one" checked disabled variant="primary" text="disabled" />}
{(value, change) => (
<RadioButton id="one" checked disabled variant="primary" text="disabled" onChange={change} />
)}
</State>
</Playground>

Expand Down