Skip to content

Add additional Storybook VSCode themes#1810

Merged
koesie10 merged 1 commit intomainfrom
koesie10/storybook-themes
Nov 29, 2022
Merged

Add additional Storybook VSCode themes#1810
koesie10 merged 1 commit intomainfrom
koesie10/storybook-themes

Conversation

@koesie10
Copy link
Copy Markdown
Member

This adds four new VSCode themes to Storybook which will allow us to more easily test these themes in Storybook. These themes were chosen because they are either used for accessibility (the high contrast themes) or are currently not compatible with the variant analysis UI (there are items that are not visible).

Checklist

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

This adds four new VSCode themes to Storybook which will allow us to
more easily test these themes in Storybook. These themes were chosen
because they are either used for accessibility (the high contrast
themes) or are currently not compatible with the variant analysis UI
(there are items that are not visible).
@koesie10 koesie10 requested a review from a team November 29, 2022 15:59
@koesie10 koesie10 requested a review from a team as a code owner November 29, 2022 15:59
Copy link
Copy Markdown
Contributor

@shati-patel shati-patel left a comment

Choose a reason for hiding this comment

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

Nice, thanks for adding these 🖼️

@koesie10 koesie10 merged commit e6fb480 into main Nov 29, 2022
@koesie10 koesie10 deleted the koesie10/storybook-themes branch November 29, 2022 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants