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

Chore: update pseudo classes to last spec. #2507

Merged
merged 1 commit into from Apr 21, 2017

Conversation

4 participants
@evilebottnawi
Member

evilebottnawi commented Apr 18, 2017

Which issue, if any, is this issue related to?

Just added new pseudo classes across latest spec https://drafts.csswg.org/selectors-4/

Is there anything in the PR that needs further explanation?

No, it's self explanatory.

@jeddy3

jeddy3 approved these changes Apr 18, 2017

LGTM

@hudochenkov

👍

@davidtheclark

This comment has been minimized.

Contributor

davidtheclark commented Apr 21, 2017

@hudochenkov: If a couple of you have approved a PR, please don't wait for me to merge unless you think there's something I definitely need to look at, or you'd like my input about. You're in charge now! 🤠

@hudochenkov

This comment has been minimized.

Member

hudochenkov commented Apr 21, 2017

@davidtheclark got it! :)

@hudochenkov hudochenkov merged commit 4f091cb into master Apr 21, 2017

4 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 95.484%
Details

@hudochenkov hudochenkov deleted the update-pseudo-classes branch Apr 21, 2017

@jeddy3

This comment has been minimized.

Member

jeddy3 commented Apr 23, 2017

Changelog:

  • Fixed: selector-pseudo-class-no-unknown now considers focus-ring, playing and paused to be known (#2507).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment