🌐 AI搜索 & 代理 主页
Skip to content

Conversation

@mgol
Copy link
Member

@mgol mgol commented Jun 26, 2023

Summary

Firefox 114+ no longer match on backslashes in :lang(), even when escaped. It is an intentional change as :lang() parameters are supposed to be valid BCP 47 strings. Therefore, we won't attempt to patch it. We'll keep this test here until other browsers match the behavior.

Fixes gh-5271
Ref https://bugzilla.mozilla.org/show_bug.cgi?id=1839747#c1 Ref w3c/csswg-drafts#8720 (comment)

Checklist

Firefox 114+ no longer match on backslashes in `:lang()`, even when escaped.
It is an intentional change as `:lang()` parameters are supposed to be valid
BCP 47 strings. Therefore, we won't attempt to patch it.
We'll keep this test here until other browsers match the behavior.

Fixes jquerygh-5271
Ref https://bugzilla.mozilla.org/show_bug.cgi?id=1839747#c1
Ref w3c/csswg-drafts#8720 (comment)
@mgol mgol added Needs review Tests Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. labels Jun 26, 2023
@mgol mgol added this to the 3.7.1 milestone Jun 26, 2023
@mgol mgol requested a review from timmywil June 26, 2023 20:38
@mgol mgol self-assigned this Jun 26, 2023
@mgol mgol merged commit 62b9a25 into jquery:main Jun 27, 2023
@mgol mgol removed Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. Needs review labels Jun 27, 2023
@mgol mgol deleted the disable-lang-escaped-test branch June 27, 2023 16:05
mgol added a commit that referenced this pull request Jun 27, 2023
Firefox 114+ no longer match on backslashes in `:lang()`, even when escaped.
It is an intentional change as `:lang()` parameters are supposed to be valid
BCP 47 strings. Therefore, we won't attempt to patch it.
We'll keep this test here until other browsers match the behavior.

Fixes gh-5271
Closes gh-5277
Ref https://bugzilla.mozilla.org/show_bug.cgi?id=1839747#c1
Ref w3c/csswg-drafts#8720 (comment)

(cherry picked from commit 62b9a25)
@mgol
Copy link
Member Author

mgol commented Jun 27, 2023

Landed on main in 62b9a25 and on 3.x-stable in 5aa7d93.

mgol added a commit to jquery/sizzle that referenced this pull request Sep 7, 2023
Firefox 114+ no longer match on backslashes in `:lang()`, even when escaped.
It is an intentional change as `:lang()` parameters are supposed to be valid
BCP 47 strings. Therefore, we won't attempt to patch it.
We'll keep this test here until other browsers match the behavior.

Ref jquery/jquery#5271
Ref jquery/jquery#5277
Ref https://bugzilla.mozilla.org/show_bug.cgi?id=1839747#c1
Ref w3c/csswg-drafts#8720 (comment)
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

Selector: Firefox 114 fails the ":lang respects escaped backslashes" selector test

2 participants