Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Support multiple-shadow syntax for box-shadow #511

Closed
wants to merge 1 commit into from

3 participants

@nooodl

(Hey, I needed this hard enough to make a patch for it!)

@Saturate

Looks fine to me. Hope it gets included soon!

@spladug
Owner

Closing because this has been obviated by the replacement of cssutils with tinycss2. Thanks for the contribution and sorry for it not getting merged.

@spladug spladug closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Aug 27, 2012
  1. @nooodl
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +2 −2 r2/r2/lib/cssfilter.py
View
4 r2/r2/lib/cssfilter.py
@@ -74,6 +74,7 @@
'single-text-shadow': r'({color}\s+)?{length}\s+{length}(\s+{length})?|{length}\s+{length}(\s+{length})?(\s+{color})?',
'box-shadow-pos': r'{length}\s+{length}(\s+{length})?(\s+{length})?',
+ 'box-shadow-single': '(?:({box-shadow-pos}\s+)?{color}|({color}\s+?){box-shadow-pos})',
}
custom_values = {
@@ -107,8 +108,7 @@
'text-shadow': r'none|({single-text-shadow}{w},{w})*{single-text-shadow}',
# http://www.w3.org/TR/css3-background/#the-box-shadow
- # (This description doesn't support multiple shadows)
- 'box-shadow': 'none|(?:({box-shadow-pos}\s+)?{color}|({color}\s+?){box-shadow-pos})',
+ 'box-shadow': r'none|{box-shadow-single}(,\s*{box-shadow-single})*',
}
def _build_regex_prefix(prefixes):
Something went wrong with that request. Please try again.