Compass / compass

Compass is no longer actively maintained. Compass is a Stylesheet Authoring Environment that makes your website design simpler to implement and easier to maintain.
http://compass-style.org
Other
6.72k stars 1.18k forks source link

Compass incorrectly omits the keyword "at" in radial-gradient rule #1674

Open jpuffer opened 10 years ago

jpuffer commented 10 years ago

The following SASS rule produces incorrect results: background-image(radial-gradient(center center, #ddd, transparent 100px)

RESULT:
background-image: -moz-radial-gradient(center center, #dddddd, transparent 100px);  
background-image: -o-radial-gradient(center center, #dddddd, transparent 100px);  
background-image: radial-gradient(center center, #dddddd, transparent 100px);  *WRONG

Changing the SASS rule to the following also produces incorrect results: background-image(radial-gradient(at center center, #ddd, transparent 100px)

RESULT:
background-image: -moz-radial-gradient(at center center, #dddddd, transparent 100px);  *WRONG
background-image: -o-radial-gradient(at center center, #dddddd, transparent 100px);  *WRONG
background-image: radial-gradient(at center center, #dddddd, transparent 100px); 

Expected behavior is that it should put the keyword "at" before "center center" in the non-prefixed version only - like so:

EXPECTED:
background-image: -moz-radial-gradient(center center, #dddddd, transparent 100px);  
background-image: -o-radial-gradient(center center, #dddddd, transparent 100px);  
background-image: radial-gradient(at center center, #dddddd, transparent 100px);  

This is per W3C spec, noted here: http://dev.w3.org/csswg/css-images-3/#radial-gradients

chriseppstein commented 10 years ago

@eddie-ruva any luck on this one?

eddie-ruva commented 10 years ago

@chriseppstein I didn't have time to work on it anymore during the weekend and I'm still at the same place with handling the svg. I will work on it today and I'm planning on getting the PR for tomorrow night.

chriseppstein commented 10 years ago

@eddie-ruva Thanks!