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

[CssSelector] add support for :is() and :where() #48803

Merged
merged 1 commit into from
Feb 3, 2024

Conversation

Jean-Beru
Copy link
Contributor

@Jean-Beru Jean-Beru commented Dec 28, 2022

Q A
Branch? 7.1
Bug fix? no
New feature? yes
Deprecations? no
Tickets Fix #48772
License MIT
Doc PR symfony/symfony-docs#17628

This PR adds support for :is() and :where() pseudo-classes.

@Jean-Beru Jean-Beru force-pushed the css-selector/add-is-and-where branch 2 times, most recently from d56aef3 to b71d1b6 Compare December 30, 2022 16:00
@fabpot
Copy link
Member

fabpot commented Dec 30, 2022

You should rebase instead of merging 6.3

@Jean-Beru Jean-Beru force-pushed the css-selector/add-is-and-where branch 2 times, most recently from 8872ef6 to 26eefb3 Compare December 30, 2022 16:31
@Jean-Beru
Copy link
Contributor Author

My bad, I used the GitHub UI to resolve conflicts 🤦‍♂️

src/Symfony/Component/CssSelector/CHANGELOG.md Outdated Show resolved Hide resolved
src/Symfony/Component/CssSelector/CHANGELOG.md Outdated Show resolved Hide resolved
src/Symfony/Component/CssSelector/CHANGELOG.md Outdated Show resolved Hide resolved
@Jean-Beru
Copy link
Contributor Author

@stof Reviews have been resolved. Is there anything to change ?

@nicolas-grekas nicolas-grekas modified the milestones: 6.3, 6.4 May 23, 2023
@Jean-Beru Jean-Beru force-pushed the css-selector/add-is-and-where branch from 4b70ca5 to 752f921 Compare July 27, 2023 14:07
@Jean-Beru
Copy link
Contributor Author

I rebased this PR to 6.4.

Should I keep it opened ?

@OskarStark OskarStark requested a review from fabpot July 28, 2023 05:47
@OskarStark
Copy link
Contributor

Friendly ping @stof @fabpot as you had some comments

Copy link
Member

@fabpot fabpot left a comment

Choose a reason for hiding this comment

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

Some random CS suggestions.

@@ -1,6 +1,12 @@
CHANGELOG
=========

6.4
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
6.4
7.1

@@ -1,6 +1,12 @@
CHANGELOG
=========

6.4
-----
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
-----
---

@fabpot fabpot force-pushed the css-selector/add-is-and-where branch from c483f62 to c083e1d Compare February 3, 2024 17:46
@fabpot
Copy link
Member

fabpot commented Feb 3, 2024

Thank you @Jean-Beru.

@fabpot fabpot merged commit d2d36b5 into symfony:7.1 Feb 3, 2024
4 of 9 checks passed
javiereguiluz added a commit to symfony/symfony-docs that referenced this pull request Feb 5, 2024
…eru)

This PR was submitted for the 6.4 branch but it was merged into the 7.1 branch instead.

Discussion
----------

[CssSelector] add support for :is() and :where()

Add documentation for
* symfony/symfony#48803

Commits
-------

f61c919 [CssSelector] add support for :is() and :where()
@Jean-Beru Jean-Beru deleted the css-selector/add-is-and-where branch March 21, 2024 15:58
@fabpot fabpot mentioned this pull request May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CssSelector] Add support for :is() and :where()
8 participants