before and after pseudoclasses

peterbe · peterbe · commit 60258a379607 · 2016-01-30T22:38:09.000-05:00
diff --git a/mincss/__init__.py b/mincss/__init__.py
@@ -1 +1 @@
-__version__ = '0.11.0'
+__version__ = '0.11.1'
diff --git a/mincss/processor.py b/mincss/processor.py
@@ -36,6 +36,9 @@
 MOUSE_PSEUDO_CLASSES = re.compile(
     ':(link|hover|active|focus|visited)$', re.M | re.I
 )
+BEFOREAFTER_PSEUDO_CLASSES = re.compile(
+    ':(before|after)$', re.M | re.I
+)
 
 
 EXCEPTIONAL_SELECTORS = (
@@ -480,8 +483,13 @@ def _found(self, bodies, selector):
         return r
 
     def _selector_query_found(self, bodies, selector):
-        # If the select has something like :active or :hover
-        if MOUSE_PSEUDO_CLASSES.findall(selector) or '::' in selector:
+        # If the select has something like :active or :hover,
+        # then evaluate it as if it's without that pseudo class
+        if (
+            MOUSE_PSEUDO_CLASSES.findall(selector) or
+            '::' in selector or
+            BEFOREAFTER_PSEUDO_CLASSES.findall(selector)
+        ):
             selector = selector.split(':')[0]
 
         if '}' in selector:
diff --git a/tests/before-after.html b/tests/before-after.html
@@ -0,0 +1,17 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <meta charset='utf-8'>
+    <title>test page</title>
+    <style>
+      ol li:before { content: "x"; }
+      ul li:after { content: "x"; }
+    </style>
+  </head>
+  <body>
+    <ol>
+      <li>First</li>
+      <li>Second</li>
+    </ol>
+  </body>
+</html>
diff --git a/tests/test_mincss.py b/tests/test_mincss.py
@@ -342,3 +342,12 @@ def test_complex_colons_in_selector_expression(self):
         after = p.inlines[0].after
         ok_('a[href^="javascript:"] { color: pink; }' in after)
         ok_('a[href^="javascript:"]:after { content: "x"; }' in after)
+
+    def test_before_after(self):
+        html = os.path.join(HERE, 'before-after.html')
+        url = 'file://' + html
+        p = Processor()
+        p.process(url)
+        after = p.inlines[0].after
+        ok_('ul li:after { content: "x"; }' not in after)
+        ok_('ol li:before { content: "x"; }' in after)

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-__version__ = '0.11.0'`
	`1`	`+__version__ = '0.11.1'`