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
Adding more tests to css-grid/animation #34137
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The review process for this patch is being conducted in the Chromium project.
a331d93
to
5a49bb6
Compare
I'm working on Chromium's implementation of grid-template animations. I noticed there are a few test cases missing. @BorisChiou would you mind taking a look since you've added the original ones? |
Sure. Will check this patch soon. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good to me. Thanks for adding the tests for composition.
Thank you for the review! |
5a49bb6
to
42e8640
Compare
This CL adds test cases to grid-template-rows-interpolation.html and grid-template-columns-interpolation.html, as well as new tests for composition of grid-template-columns and grid-template-rows. Bug: 759665 Change-Id: Ibf77a550964732aa12773342a117b96529b1215c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3591246 Commit-Queue: Ana Sollano Kim <ansollan@microsoft.com> Reviewed-by: Daniel Libby <dlibby@microsoft.com> Cr-Commit-Position: refs/heads/main@{#1007077}
42e8640
to
f56fe4d
Compare
This CL adds test cases to grid-template-rows-interpolation.html and
grid-template-columns-interpolation.html, as well as new tests for
composition of grid-template-columns and grid-template-rows.
Bug: 759665
Change-Id: Ibf77a550964732aa12773342a117b96529b1215c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3591246
Commit-Queue: Ana Sollano Kim <ansollan@microsoft.com>
Reviewed-by: Daniel Libby <dlibby@microsoft.com>
Cr-Commit-Position: refs/heads/main@{#1007077}