-
Notifications
You must be signed in to change notification settings - Fork 18
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
feat(Accordion): Clockface 4 Updates #859
Conversation
5117741
to
27afbad
Compare
@@ -106,7 +96,7 @@ export const AccordionRoot = forwardRef<AccordionRef, AccordionProps>( | |||
const contextState = { | |||
isExpanded, | |||
setExpanded, | |||
iconPlacementPosition: hasBody ? iconPlacementPosition : Direction.None, | |||
iconPlacementPosition: hasBody ? Direction.Left : Direction.None, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We want to force left placement of icons for more consistency.
@@ -283,9 +281,6 @@ accordionStories.add( | |||
style={{justifyContent: 'none', alignItems: 'start', display: 'block'}} | |||
> | |||
<Accordion | |||
iconPlacement={ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
delete that option from storybook
27afbad
to
c9726e3
Compare
@@ -411,9 +400,14 @@ accordionStories.add( | |||
</Accordion.AccordionHeader> | |||
<Accordion.AccordionBodyItem> | |||
<span> | |||
This is your detailed body. This is your detailed body. This is |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found this annoying, so changed it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lorem ipsum is a pretty standard replacement.
c9726e3
to
f0f85f9
Compare
1. Colors and Icon placement 2. Fixed the icon bug for when accordion opens/closes (it didn't change icons) 3. Updated storybook
f0f85f9
to
642d8a7
Compare
&--no-body { | ||
margin-left: 25px; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Part of #6090 |
Closes: #822
Accordion updates for Clockface 4 to match figma.
Screen.Recording.2022-10-05.at.11.11.18.PM.mov