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

keyframes issue #129

Open
alexandru-calinoiu opened this issue Aug 28, 2014 · 2 comments
Open

keyframes issue #129

alexandru-calinoiu opened this issue Aug 28, 2014 · 2 comments

Comments

@alexandru-calinoiu
Copy link

Hi,

I have the follwoing less code that uses lesshat:

// FadeIn
.fadeIn(@delay: 0) {
    .animation-delay(@delay);
    .animation-name(fadeIn);
    .animation-duration(1.5s);
    .animation-fill-mode(forwards);
    .transition-timing-function(ease-in-out);
}

.keyframes(~'fadeIn, 0% { transform: scale(0); opacity: 0.0; } 60% { transform: scale(1.1); } 80% { transform: scale(0.9); opacity: 1; } 100% { transform: scale(1); opacity: 1; }');

When compiling this I will get the following error:

ExecJS::ProgramError 

Unclosed block
@-o-keyframes fadeIn{ 0% { -o-transform: scale(0); opacity: 0.0; }}
@keyframes fadeIn{ 0% {-webkit-transform: scale(0);-moz-transform: scale(0);-ms-transform: scale(0);transform: scale(0); opacity: 0.0; };
^
}
@tjwudi
Copy link

tjwudi commented Sep 17, 2014

I just got this error too

@davipt
Copy link

davipt commented Jun 12, 2015

This major issue is now also affecting Safari on OS X 10.11 and iOS 9 betas.
The issue is not the extra semicolon at the end of the @Keyframes, but the fact that the line is not terminated by the "}" on a contiguous line, but on the line below.
If you try to move the last "}" up, it starts working with Safari. I assume the same will happen with the HTC. Even the less hat code mentions that the "@*keyframes must be in a single line".
I've added a pull requests that moves the "}" up, by having to create a fake line below it. Please see it below.

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

No branches or pull requests

3 participants