search

sasstools / scss-tokenizer

A tokenzier for Sass' SCSS syntax
MIT License
24 stars 22 forks source link

Interpolation closing brace is not tokenized #20

Closed mouhannad-sh closed 5 years ago

mouhannad-sh commented 5 years ago

Hi I've tried the following example:

tokenize(' #{$var +20} 10')

the output is 

[ [ 'space', ' ' ],
  [ 'startInterpolant', '#{', 1, 3 ],
  [ '$', '$', 1, 4 ],
  [ 'ident', 'var', 1, 5, 1, 7 ],
  [ 'space', ' ' ],
  [ '+', '+', 1, 9 ],
  [ 'number', '20', 1, 10, 1, 11 ], // closing brace should be after this
  [ 'space', ' ' ],
  [ 'number', '10', 1, 14, 1, 15 ] ]

as you can see there's a startInterpolant token but the output doesn't indicate anything about the the closing brace.

Am I missing anything ? is there a different way to handle this syntax #{ } ?

Thanks

xzyfer commented 5 years ago

Looks like you found a bug. I'll look into it this week

On Sun., 2 Dec. 2018, 11:15 pm Nedo <notifications@github.com wrote:

Hi I've tried the following example:

tokenize(' #{$var +20} 10')

the output is

[ [ 'space', ' ' ], [ 'startInterpolant', '#{', 1, 3 ], [ '$', '$', 1, 4 ], [ 'ident', 'var', 1, 5, 1, 7 ], [ 'space', ' ' ], [ '+', '+', 1, 9 ], [ 'number', '20', 1, 10, 1, 11 ], // closing brace should be after this [ 'space', ' ' ], [ 'number', '10', 1, 14, 1, 15 ] ]

as you can see there's a startInterpolant token but the output doesn't indicate anything about the the closing brace.

Am I missing anything ? is there a different way to handle this syntax #{ } ?

Thanks

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/sasstools/scss-tokenizer/issues/20, or mute the thread https://github.com/notifications/unsubscribe-auth/AAjZWNIVGUzkWbvEA-g7x1TMV0m0Q8uKks5u08RbgaJpZM4Y9X9v .

xzyfer commented 5 years ago

Fixed in 0.4.0