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

feat(utils/matches): support selectors level 4 :not and :is #2742

Merged
merged 4 commits into from
Jan 11, 2021

Conversation

straker
Copy link
Contributor

@straker straker commented Jan 7, 2021

Been wanting to do this for awhile now so decided to take some time to do it. We can now support the Selectors Level 4 :is pseudo selector (which allows us to simplify some heading selecting) and allows :not to support more than one selector, which also helps us shorten and simplify selectors.

Since we use utils/matches for everything and we don't go through the DOM selector API, we don't have to worry about browser support since we match everything virtually.

@straker straker requested a review from a team as a code owner January 7, 2021 23:37
@straker straker changed the title feat(utils/matches): support selector-v4 :not and :is feat(utils/matches): support selectors level 4 :not and :is Jan 7, 2021
Copy link
Contributor

@WilcoFiers WilcoFiers left a comment

Choose a reason for hiding this comment

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

Minor test changes

test/core/utils/matches.js Outdated Show resolved Hide resolved
test/core/utils/matches.js Show resolved Hide resolved
straker and others added 2 commits January 8, 2021 08:07
Co-authored-by: Wilco Fiers <WilcoFiers@users.noreply.github.com>
@straker straker merged commit 21d9b0e into develop Jan 11, 2021
@straker straker deleted the selectors-v4 branch January 11, 2021 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants