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

[Todo] :not([data-theme-effect]) -> :root:not([data-theme-effect]) #244

Open
zaydek opened this issue Jan 3, 2021 · 0 comments
Open

[Todo] :not([data-theme-effect]) -> :root:not([data-theme-effect]) #244

zaydek opened this issue Jan 3, 2021 · 0 comments
Labels
bug Something isn't working zaydek

Comments

@zaydek
Copy link
Owner

zaydek commented Jan 3, 2021

It would be better if we could just target :not(...) but we need to make sure this selector is actually valid first. If memory serves, it’s not.

@zaydek zaydek added bug Something isn't working zaydek labels Jan 3, 2021
@zaydek zaydek changed the title [Regression] :not([data-theme-effect]) -> :root:not([data-theme-effect]) [Fixme] :not([data-theme-effect]) -> :root:not([data-theme-effect]) Jan 3, 2021
@zaydek zaydek changed the title [Fixme] :not([data-theme-effect]) -> :root:not([data-theme-effect]) [Todo] :not([data-theme-effect]) -> :root:not([data-theme-effect]) Jan 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working zaydek
Projects
None yet
Development

No branches or pull requests

1 participant