-
Notifications
You must be signed in to change notification settings - Fork 29
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
Rework flexible/general
stories
#12464
base: main
Are you sure you want to change the base?
Conversation
Size Change: +2 B (0%) Total Size: 923 kB ℹ️ View Unchanged
|
de127b7
to
e4592ad
Compare
@@ -154,7 +154,7 @@ const HorizontalDivider = () => ( | |||
${from.desktop} { | |||
width: 140px; | |||
} | |||
margin-top: 12px; | |||
margin-top: ${space[3]}px; |
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.
not related, just ensuring we use source spacing where possible
@@ -101,8 +101,8 @@ export const OneCardLayout = ({ | |||
imagePositionOnMobile, | |||
supportingContentAlignment, | |||
} = determineCardProperties( | |||
card.boostLevel, | |||
card?.supportingContent?.length ?? 0, | |||
card.boostLevel ?? 'default', |
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.
Refactoring: cards with unspecified boostLevel
are considered to be unboosted ie "default"
@@ -19,7 +19,7 @@ type Props = { | |||
absoluteServerTimes: boolean; | |||
}; | |||
|
|||
type boostProperties = { | |||
type BoostProperties = { |
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.
192d04a
to
73c16f9
Compare
What does this change?
Updates the Storybook stories for the
FlexibleGeneral
component to better demonstrate the variations in designs for various card configurations in as few stories as possible.We now create a list of standard cards with defined variations in design so that we can visualise all the design quirks in one story. Each story now represents variations to the splash (sublinks + boost level) and not variations in the standard cards
Also contains other semi-related refactoring work to
FlexibleSpecial
andFlexibleGeneral
Why?
Part of this Trello ticket
We can use this to better visualise and document which design variations and configuration options there are with the
FlexibleGeneral
container as well as showing what design features are currently missing.See Chromatic diff for better visualisation of changes.
Screenshots