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 css/css-cascade/scope-starting-style.html from @starting-style & transition-behavior focus-area #641

Closed
dholbert opened this issue Feb 29, 2024 · 6 comments · Fixed by web-platform-tests/wpt-metadata#5716
Labels
focus area: @starting-style & transition-behavior test-change-proposal Proposal to add or remove tests for an interop area

Comments

@dholbert
Copy link

dholbert commented Feb 29, 2024

Test List

https://wpt.fyi/results/css/css-cascade/scope-starting-style.html

Rationale

This test uses the @scope CSS feature (and appears to require this feature in order to pass).

@scope isn't inherently part of or related to @starting-style & transition-behavior. It's nice to test the interaction, but it doesn't make sense to have @scope as an implicit implementation requirement for another interop-2024 focus area.

(Note that @scope isn't broadly available yet -- Safari only has it enabled in TP, and Firefox doesn't have it implemented at all yet -- we've got someone looking at implementing, but nonetheless we'd prefer not to tie it to this mostly-unrelated focus area.)

@dholbert
Copy link
Author

(Note: it seems this test was included in this focus area simply because the proposal in #582 had a proposed-test-list that matched any WPT with starting-style in the name, which includes this one.)

@nt1m
Copy link
Member

nt1m commented Mar 1, 2024

seems fine

@dholbert
Copy link
Author

dholbert commented Mar 6, 2024

Thanks @nt1m!

@josepharhar is this fine with you? (removing scope-starting-style.html from the starting-style interop focus area here)

@josepharhar
Copy link

is @scope part of interop2024? If it is, then this test should be part of something in interop2024, right? If not then it seems fine to remove

@nt1m
Copy link
Member

nt1m commented Mar 6, 2024

@scope is not part of Interop 2024

@dholbert
Copy link
Author

dholbert commented Mar 6, 2024

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
focus area: @starting-style & transition-behavior test-change-proposal Proposal to add or remove tests for an interop area
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants