transition bug #805

Closed
chrisdrackett opened this Issue Mar 23, 2012 · 7 comments

Comments

Projects
None yet
5 participants
@chrisdrackett

I just updated compass and I've run into the following:

I have my transition properties defined in a variables file:

$default-transition-property : all          !default
$default-transition-duration : .15s         !default
$default-transition-function : ease-in-out  !default
$default-transition-delay    : false        !default

then I generally just add +transition() to a class that I want to animate. However, I see that this now outputs:

-webkit-transition: all, 0.3s, ease-in-out;
-moz-transition: all 0.3s ease-in-out;
-ms-transition: all 0.3s ease-in-out;
-o-transition: all 0.3s ease-in-out;
transition: all, 0.3s, ease-in-out;

I believe that the commas are causing issues. Maybe I'm missing something?

@chriseppstein

This comment has been minimized.

Show comment Hide comment
@chriseppstein

chriseppstein Mar 23, 2012

Member

@chrisdrackett what version of compass?

Member

chriseppstein commented Mar 23, 2012

@chrisdrackett what version of compass?

@chrisdrackett

This comment has been minimized.

Show comment Hide comment
@chrisdrackett

chrisdrackett Mar 23, 2012

0.12.1

0.12.1

@scottdavis

This comment has been minimized.

Show comment Hide comment
@scottdavis

scottdavis Mar 24, 2012

Member

werid

Member

scottdavis commented Mar 24, 2012

werid

@scottdavis

This comment has been minimized.

Show comment Hide comment
@scottdavis

scottdavis Mar 24, 2012

Member

@doctyper was the last one to work in this file maybe he can comment?

Member

scottdavis commented Mar 24, 2012

@doctyper was the last one to work in this file maybe he can comment?

@chriseppstein

This comment has been minimized.

Show comment Hide comment
@chriseppstein

chriseppstein Mar 24, 2012

Member

I want to do a 0.12.2 release this weekend. This needs to be addressed for that release.

Member

chriseppstein commented Mar 24, 2012

I want to do a 0.12.2 release this weekend. This needs to be addressed for that release.

doctyper added a commit to doctyper/compass that referenced this issue Mar 24, 2012

@doctyper

This comment has been minimized.

Show comment Hide comment
@doctyper

doctyper Mar 25, 2012

Contributor

Bump @chriseppstein. Don't want to miss your .2 release.

Contributor

doctyper commented Mar 25, 2012

Bump @chriseppstein. Don't want to miss your .2 release.

@mirisuzanne

This comment has been minimized.

Show comment Hide comment
@mirisuzanne

mirisuzanne Apr 11, 2012

Member

closed by 0c13d98

Member

mirisuzanne commented Apr 11, 2012

closed by 0c13d98

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment