Skip to content

Conversation

cderv
Copy link
Collaborator

@cderv cderv commented Sep 17, 2024

This is a follow up on b7a5449 which fixed that for the light theme

We do apply the same fix for dark theme.

fixes #10817

Now that playwright test are easier to write, this adds a test to check the CSS value defined is the correct one from user's custom theme file

@cderv
Copy link
Collaborator Author

cderv commented Sep 17, 2024

Failure in playwright test don't make the test fails and CI erroring. So I need to improve that... 🤔

Copy link
Collaborator

@cscheid cscheid left a comment

Choose a reason for hiding this comment

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

Looks good aside from maybe a stray extra file.

@cderv cderv merged commit fa772d3 into main Sep 17, 2024
47 checks passed
@cderv cderv deleted the fix/dark-theme-layering branch September 17, 2024 19:17
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.

Dark theme doesn't respect the layering order of the stylesheets

2 participants