899097 - Update slowparse to deal with @keyframes

Status: RESOLVED FIXED

(Webmaker Graveyard :: Thimble, defect)

Description

[aklee237]

Attachment: spaceinvaders.html

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/28.0.1500.72 Safari/537.36

Steps to reproduce:

This is my CSS

@keyframes animation1
    {
      /*Change where the cloud starts */      
0% { left: 260px; top: -10%; }
      /* Change where it ends */
100% { left: 260px; top: 100%; }
}
    


Actual results:

Thimble gives an error message stating

"Missing block closer or next property:value; pair following @keyframes animation1."


Expected results:

it should just work

Comment 1

[Pomax]

This is a SlowParse issue, caused by the nessted {} blocks. Its parsing rules will have to be updated for allowing nesting in specific @syntax contexts

Comment 2

[aklee237]

thanks for the reply,

anything i can do to help out? https://github.com/mozilla/slowparse

I'm trying to get this ready for the Hive NYC maker party on Thur

Comment 3

[Pomax]

if you want to give updating it a shot: _parseDeclaration needs to become aware of the extra rules involved in parsing @keyframe's nested blocks, rather than parsing individual properties. We're hampered a little here because slowparse is effectivly a DFA and runs through the sequence

parseSelector -> parseDeclaration -> parseProperty -> parseValue -> (parseDeclaration or parseSelector)

so as a closed loop, busting this open with special rules for @keyframes will either have to set up a second loop,

1. parseSelector -> parseDeclaration -> (parseProperty OR parseKeyframeRules)
2a. parseProperty -> parseValue -> (parseDeclaration or parseSelector)
2b. parseKeyframeRules -> parseIndicator -> parseBlock

where parseBlock somehow does the same as the property/value chain without ending up calling parseDeclaration, because then we end up in the wrong route.

This got shelved quite a while ago because it was a fairly hard problem to solve cleanly, so if you want to try to add this in, you would be a true hero, although I don't know if there would be enough time to make it work before Thursday while keeping it test driven (then again, sometimes one day is plenty if you have a brainwave that lets you write exactly the code you needed)

Comment 4

[aklee237]

thanks for the tips. won't get to it in time for this Thur but I do want to solve this in time for the NYC Maker Faire

Comment 5

[Pomax]

unassigning pending reprioritisation

Comment 7

[Pomax]

Attachment: https://github.com/mozilla/thimble.webmaker.org/pull/360

if you think of pass/fail tests, please let me know and we can add them.

Comment 8

[Ali Al Dallal [:alicoding]]

Comment on attachment 8377830 [details] [review]
https://github.com/mozilla/thimble.webmaker.org/pull/360

All tests passed really good patch!