Conversation
|
|
||
| // match a property name (any-possible_name) | ||
| const matchDeclaration = /^([\w-]+)\s*:\s*([\W\w]+)\s*$/; | ||
| const matchDeclaration = /^["']?([\w-]+)["']?\s*:\s*([\W\w]+)\s*$/; |
There was a problem hiding this comment.
quoted keys where not supported
| const getEachOpts = (node, opts) => { | ||
| const params = node.params.split(matchInOperator); | ||
| const args = (params[0] || '').trim().split(' '); | ||
| const args = (params[0] || '').trim().split(/[\s,]+/); |
There was a problem hiding this comment.
@each $value, $key in $arrayTest {
support for sass syntax -> comma
| const reverseParams = incname && !Array.isArray(list) | ||
| if (reverseParams) { | ||
| return { incname: varname, varname: incname, list }; | ||
| } |
There was a problem hiding this comment.
in case of object if 2 arguments are passed, its key, value
| ."1"-arrayTest { | ||
| content: "1"; | ||
| } | ||
| ."2"-arrayTest { |
There was a problem hiding this comment.
this idffers from sass, as sass removes "', not sure if you want me to remove it to
removal of quotes is only present if string is \w_ from what i observed
armano2
force-pushed
the
fix/each-improvments
branch
from
f561999 to
5b7ae89
Compare
|
@Nunnius for time beeing this and few other fixes are released as |
Do you plan to update the README.md of your project to explain it’s a fork, and that it can be downloaded from |
3 participants
Adds missing tests and improves support to
@each objectfixes #88, fixes #91, fixes #59