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

False positive on transition: grid and transition: grid-* #197

Open
yassh opened this issue Oct 28, 2024 · 2 comments
Open

False positive on transition: grid and transition: grid-* #197

yassh opened this issue Oct 28, 2024 · 2 comments

Comments

@yassh
Copy link

yassh commented Oct 28, 2024

Unlike the Can I use database, doiuse warns against the usage of transition: grid and transition: grid-* like this:

main.css:2:1: CSS Grid animation not supported by: Edge (129,130), Chrome (129,130), Chrome for Android (129) (css-grid-animation)
main.css:3:1: CSS Grid animation not supported by: Edge (129,130), Chrome (129,130), Chrome for Android (129) (css-grid-animation)
main.css:4:1: CSS Grid animation not supported by: Edge (129,130), Chrome (129,130), Chrome for Android (129) (css-grid-animation)
main.css:5:1: CSS Grid animation not supported by: Edge (129,130), Chrome (129,130), Chrome for Android (129) (css-grid-animation)
main.css:7:1: CSS Grid animation not supported by: Edge (129,130), Chrome (129,130), Chrome for Android (129) (css-grid-animation)
main.css:8:1: CSS Grid animation not supported by: Edge (129,130), Chrome (129,130), Chrome for Android (129) (css-grid-animation)
main.css:9:1: CSS Grid animation not supported by: Edge (129,130), Chrome (129,130), Chrome for Android (129) (css-grid-animation)

main.css:

.test {
  transition: grid 1s;
  transition: grid-template 1s;
  transition: grid-template-rows 1s;
  transition: grid-template-columns 1s;

  transition: grid-template-areas 1s;
  transition: grid-auto-rows 1s;
  transition: grid-auto-columns 1s;
}
@clshortfuse clshortfuse added bug and removed bug labels Oct 29, 2024
@clshortfuse
Copy link
Collaborator

I don't see the problem. Animation of grid is done with transition property. If anything it's missing the animation: rule as well.

grid is a shorthand for the other properties.

@yassh
Copy link
Author

yassh commented Oct 31, 2024

I found that this issue was caused by errors in the caniuse database.

I submitted a pull request to fix the errors, and it was merged:
Fyrd/caniuse#7196

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants