Skip to content
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

Makes sure that the property of a style is not an url #78

Merged
merged 2 commits into from
Apr 13, 2017

Conversation

jfabre
Copy link
Contributor

@jfabre jfabre commented Apr 10, 2017

Hello,

just had this issue that made me sweat for a while today.
Please let me know if I need to do anything else to get this PR merged.

Commit msg

I actually got a bug with background-image property
pointing to cloudfront with this url:
https://d3r0d0kb2vhuxd.cloudfront.net

The viewportUnitExpression regex matched the url
and the url got rewritten.

This is a fix to avoid that.

@rodneyrehm
Copy link
Owner

interesting. thanks for figuring this out!

if you could rename url to urlExpression (to fit with the others) and rename the commit message to fix(polyfill): do not consider URLs viable values for running the polyfill.

…yfill.

I actually got a bug with background-image property
pointing to cloudfront with this url:
  https://d3r0d0kb2vhuxd.cloudfront.net

The viewportUnitExpression regex matched the url
and the url got rewritten.

This is a fix to avoid that.
@jfabre
Copy link
Contributor Author

jfabre commented Apr 13, 2017

ping! not sure why there's a conflict now.

@rodneyrehm
Copy link
Owner

rodneyrehm commented Apr 13, 2017

the file was reindented. I resolved the conflict using the web editor (did not know this github feature existed...) thx!

@rodneyrehm rodneyrehm merged commit 457e53a into rodneyrehm:master Apr 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants