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.

Sign up for GitHub

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

Jump to bottom

[selectors] Simplify @supports check in :focus-visible tests #28504

Merged
merged 1 commit into from
Apr 15, 2021
Merged

[selectors] Simplify @supports check in :focus-visible tests #28504

merged 1 commit into from
Apr 15, 2021

Conversation

mrego
Copy link
Member

@mrego mrego commented Apr 15, 2021

Please @emilio review this. This is the change you suggested in the review of #27806. Thanks!

@mrego mrego requested a review from emilio April 15, 2021 10:19
@delan delan self-requested a review April 15, 2021 11:33
delan
delan approved these changes Apr 15, 2021
View reviewed changes
Copy link
Member

@delan delan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, seems pretty straightforward.

@mrego mrego merged commit 19445e7 into web-platform-tests:master Apr 15, 2021
@mrego mrego deleted the focus-visible-supports branch April 15, 2021 11:34
@mrego mrego mentioned this pull request Apr 15, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@delan delan delan approved these changes

@emilio emilio Awaiting requested review from emilio

@fantasai fantasai Awaiting requested review from fantasai

@frivoal frivoal Awaiting requested review from frivoal

@plinss plinss Awaiting requested review from plinss

@tabatkins tabatkins Awaiting requested review from tabatkins

Assignees

@frivoal frivoal

Labels
selectors
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mrego @delan @frivoal