-
Notifications
You must be signed in to change notification settings - Fork 2
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
Does not recognize multiple background-images properly #13
Comments
yikes, maybe we should just move towards using this could probably be fixed up, but I wonder if it's worth it if these issues keep coming up. |
Yeah, I totally see that being a valuable way to offload this effort. fwiw, I've also considered adopting |
yah i'm +1 to this change. originally it was to speed up the parsing, but i didn't realize the complexity and developer cost plain regexps would bring. |
Cool, would you mind moving this to the |
done, added you on npm as well |
When using multiple backgrounds, this module does not parse
url()
declarations after the first one properly:The first one (
./logo.png
) gets parsed correctly, but the 2nd (./caret.png
) is not detected. (and is not rewritten by the consumer)The text was updated successfully, but these errors were encountered: