sapegin / grunt-webfont

SVG to webfont converter for Grunt
MIT License
1.11k stars 210 forks source link

Set value for fontSrcSeparators directly in options #316

Open thybzi opened 8 years ago

thybzi commented 8 years ago

In source code of tasks/util/util.js I can find:

/**
 * @font-face’s src parts seperators.
 * @type {Object}
 */
exports.fontSrcSeparators = {
    _default: ',\n\t\t',
    styl: ', '
};

So, if I want to set this separator to ', ' I need to set stylesheet to styl (not so evident, uh?) (And if I want to set this to ',\n    ', my only option is to fork and change source code ;)

So my proposal is to make a separate option to override this separator value directly.

Thank you!

sapegin commented 8 years ago

Why do you want to change it? It’s generated file, code style doesn’t matter here. The only reason for this options is that new lines didn’t work in Stylus. Maybe they work now and this const can be removed from code.

thybzi commented 8 years ago

Ok, I may try writing this. Keep in touch then and wait for pull request!