alyssais/Sass-Web-Fonts

if only one font with spaces is included (like 'Maven Pro') it crashes the parser

w4tsn opened this issue · 2 comments

If there is only one Font which has white spaces in it, the parser handles it as two fonts. That could be caused by the scss language itself which interprets both strings comma-separated and white-space-separated as list.

e.g. If there is a list of two Fonts: Maven Pro, Josefin Sans then ',' is the separator of the list. But if there is only Maven Pro something tries to see it as a list of two strings.
'Maven' and 'Pro' separated with a white-space.

I hope my explanation is understandable and someone if not me can solve this bug ;)

I also have this issue, sadness.

This was due to a lack of string processing in Sass 3.2. I rewrote the mixin for Sass 3.3 and it should no longer have problems with these. Could you give the new version a try and let me know how it goes?

I'm marking this as closed for now since I believe it to be fixed, but feel free to reopen it if the issue is still there.

Note: I changed the interface a bit in the rewrite, so have a read over the documentation before you try out the new version.