You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Test change proposal: remove at-property-shadow.html #673
Also, I vaguely remember @astearns concluding (at some point during CSS Day) that we might want to go back and
reconsider how it should work in the first place.
Under the above circumstances, I probably would have objected to including the test in the first place.
Note that the proposal is to remove the test from the interop set, not to actually delete it.
FWIW I think if the outcome here is spec changes that lead to everyone agreeing the test is correct as-is, and therefore continuing to pass it, I doubt there'd be many objections to re-adding the test to Interop.
Reacted by Emilio Cobos Álvarez, Alan Stearns and andruud
Activity
emilio commentedon Jul 8, 2024
cc @andruud @nt1m
[-]Test change proposal: at-property-shadow.html[/-]nt1m commentedon Jul 8, 2024
Would be better if we can fix it, but open to removing if we can't.
andruud commentedon Jul 29, 2024
(I was OOO for a bit).
Yes, please go ahead and remove it.
reconsider how it should work in the first place.
Under the above circumstances, I probably would have objected to including the test in the first place.
astearns commentedon Jul 29, 2024
It’s a good test to have, we just might need to change the expected result (as I did to match the current spec).
I’d prefer to keep the test, open an issue on the spec, then change the test if we decide to make changes to the spec.
jgraham commentedon Jul 29, 2024
Note that the proposal is to remove the test from the interop set, not to actually delete it.
FWIW I think if the outcome here is spec changes that lead to everyone agreeing the test is correct as-is, and therefore continuing to pass it, I doubt there'd be many objections to re-adding the test to Interop.
Update META.yml
Update META.yml