Merge pull request #95 from Gallaecio/fix-scope-not-first

Support :scope after a comma delimiter
scrapy · Oct 24, 2022 · 5c2719d · 5c2719d
2 parents 1514eff + aa4ff01
commit 5c2719d
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/cssselect/parser.py b/cssselect/parser.py
@@ -625,6 +625,12 @@ def parse_simple_selector(stream, inside_negation=False):
                     if not (
                         len(stream.used) == 2
                         or (len(stream.used) == 3 and stream.used[0].type == "S")
+                        or (len(stream.used) >= 3 and stream.used[-3].is_delim(","))
+                        or (
+                            len(stream.used) >= 4
+                            and stream.used[-3].type == "S"
+                            and stream.used[-4].is_delim(",")
+                        )
                     ):
                         raise SelectorSyntaxError(
                             'Got immediate child pseudo-element ":scope" '

diff --git a/tests/test_cssselect.py b/tests/test_cssselect.py
@@ -230,6 +230,14 @@ def test_pseudo_repr(css):
             ("CombinedSelector[Pseudo[Element[*]:scope] > Element[div]]", None),
             ("CombinedSelector[Element[foo] <followed> Element[bar]]", None),
         ]
+        assert parse_pseudo("foo bar, :scope > div") == [
+            ("CombinedSelector[Element[foo] <followed> Element[bar]]", None),
+            ("CombinedSelector[Pseudo[Element[*]:scope] > Element[div]]", None),
+        ]
+        assert parse_pseudo("foo bar,:scope > div") == [
+            ("CombinedSelector[Element[foo] <followed> Element[bar]]", None),
+            ("CombinedSelector[Pseudo[Element[*]:scope] > Element[div]]", None),
+        ]
         assert parse_pseudo("foo:before, bar, baz:after") == [
             ("Element[foo]", "before"),
             ("Element[bar]", None),