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

Remove grid-animation #2441

Open
alisonmaher opened this issue Dec 12, 2024 · 3 comments
Open

Remove grid-animation #2441

alisonmaher opened this issue Dec 12, 2024 · 3 comments
Labels
feature definition Creating or defining new features or groups of features.

Comments

@alisonmaher
Copy link

Specification

N/A (this is request to remove)

Description

Grid-animation was treated as a separate feature from Grid because it shipped later on, but at this point, we can consider this captured within the Grid feature all up (and all tests for grid-animation live under css-grid already).

As such, I think we should remove the existing entry for grid-animation: https://webstatus.dev/features/grid-animation

Documentation

N/A (this is a request to remove)

Browser support

No response

@alisonmaher alisonmaher added the feature definition Creating or defining new features or groups of features. label Dec 12, 2024
@jamesnw
Copy link
Collaborator

jamesnw commented Dec 13, 2024

I find it useful to know that grid-animation is not yet widely available, and think it's still important to communicate that different support story to users. We often use the group attribute to relate later additions like this, and grid and grid-animation are both are in the same grid group.

I am curious about your use case, though- are you doing some reporting around tests related to the feature ids?

@alisonmaher
Copy link
Author

There was a request to add feature metric tracking for all web features listed in this repo. However, Grid is already being tracked, and grid-animation can be considered tracked under Grid all up. This request is to simplify feature tracking by removing this category since grid-animation is captured within Grid and its test suite already.

@captainbrosset
Copy link
Contributor

We are 4 months away from grid animation becoming baseline high. When it does, it makes a lot of sense to merge the two features. Before this, I tend to agree with James.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature definition Creating or defining new features or groups of features.
Projects
None yet
Development

No branches or pull requests

3 participants