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

ci(theme): swap default themes to eui-light #2186

Merged
merged 4 commits into from
Oct 6, 2023

Conversation

nickofthyme
Copy link
Collaborator

@nickofthyme nickofthyme commented Oct 6, 2023

Summary

Changes default e2e and storybook theme with eui themes in preparation for migration to eui theme as default.

Details

This PR is meant to act as a simple switch to the eui theme in all VRT screenshots with no other changes. Later in #2175 we will merge the default chart theme with the eui theme.

Checklist

  • The proper chart type label has been added (e.g. :xy, :partition)
  • The proper feature labels have been added (e.g. :interactions, :axis)
  • The proper documentation and/or storybook story has been added or updated

@nickofthyme nickofthyme requested a review from markov00 October 6, 2023 05:10
@nickofthyme nickofthyme added the :ci label Oct 6, 2023
@nickofthyme
Copy link
Collaborator Author

buildkite update screenshots

@nickofthyme nickofthyme added :all Applies to all chart types :docs Anything related to documentation, API, storybook and removed :all Applies to all chart types labels Oct 6, 2023
@nickofthyme
Copy link
Collaborator Author

buildkite update screenshots

@nickofthyme nickofthyme changed the title ci(theme): swap default themes to eui themes ci(theme): swap default themes to eui-light Oct 6, 2023
@nickofthyme nickofthyme merged commit 9310366 into elastic:main Oct 6, 2023
11 checks passed
@nickofthyme nickofthyme deleted the set-default-theme branch October 6, 2023 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:ci :docs Anything related to documentation, API, storybook
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant