test(documentation): added basic e2e tests for the new palette implementation #5338
Conversation
|
|
|
Related Previews |
| @@ -1 +1,3 @@ | |||
| <script src="/assets/scripts/storybook-preview-events.js"></script> | |||
| <body data-color-scheme="light"> | |||
There was a problem hiding this comment.
light dark should be the default without having the specify a data-color-scheme attribute. Do you know why it is necessary here?
| scheme => html` <div data-color-scheme="${scheme}" class="p-16 palette-default"> | ||
| ${renderFn(scheme)} | ||
| </div>`, | ||
| scheme => html` <div data-color-scheme="${scheme}" class="p-16">${renderFn(scheme)}</div>`, |
There was a problem hiding this comment.
It would be good to test how the palettes behave without a defined data-color-scheme, I have seen this article about dark mode in Cypress: https://www.cypress.io/blog/test-your-web-app-in-dark-mode
📄 Description
This PR implements some basic e2e test for the new palettes integration.
However, they are not complete, but are meant to stabilize the current implementation so that we can figure out if anything goes horribly wrong during further development.
📝 Checklist