search

jubalm / grunt-fontello

grunt task for downloading fonts from fontello.com
MIT License
45 stars 25 forks source link

Change space variable from '\t' to ' ' #31

Closed paweltin closed 7 years ago

paweltin commented 8 years ago

Fixes #30

ryuran commented 8 years ago

This fix will change \t to `for people who use\t`. So it's Not the solution. I don't understand why grunt fontello rewrite the config file.

spAnser commented 7 years ago

I just commented out the line he modified. It is only used for storing your session id but it is a step that can probably save a little time downloading the files but it is also unnecessary. You can just start a new session every time without problems. So commenting out that line will do just that.

Also I think he changed it to spaces because that is what the config.json downloaded from Fontello uses itself.

ericcornelissen commented 7 years ago

Closing this PR since the issue has been solved by #38