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

Support previewing a design directly #514

Merged
merged 1 commit into from
Oct 13, 2023
Merged

Support previewing a design directly #514

merged 1 commit into from
Oct 13, 2023

Conversation

tjmw
Copy link
Member

@tjmw tjmw commented Oct 13, 2023

What does this change?

Adds live previewing to the banner design page, using a hardcoded variant configuration. In a future PR I'd like to add limited configuration of the variant (toggling the ticker for example) but for now I think this is a good first step.

Screenshot 2023-10-13 at 09 38 39

How to test

Run locally or on CODE. Click that live preview button!

How can we measure success?

Have we considered potential risks?

Images

Accessibility

@tjmw tjmw force-pushed the tw/preview-design branch from d547a3a to f88b8c4 Compare October 13, 2023 08:48
Copy link
Member

@tomrf1 tomrf1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

No support for showing a ticker yet.
@tjmw tjmw force-pushed the tw/preview-design branch from f88b8c4 to b9d02c9 Compare October 13, 2023 08:53
@tjmw tjmw merged commit bc56018 into main Oct 13, 2023
@tjmw tjmw deleted the tw/preview-design branch October 13, 2023 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants