Use without material icon font

[ramijarrar]

Can you make the zodiase:material-design-icons-fonts dependency optional? We are using an SVG sprite to load our custom icon-set (as well as a small subset of the material icons) so the icon font from that package is unnecessary.

[Zodiase]

This was part of the plan when I was trying to support configurable theme and SCSS but it got a bit too complicated for the time I had. I'll look into this some time soon.

[ramijarrar]

Would it not just be a matter of removing this line/wrapping in a conditional based on settings? It seems like we already have good separation?

[Zodiase]

That would be the solution except in package.js I can't reliably access the configs file thus there's nothing to be used as the condition. See the old code https://github.com/Zodiase/meteor-mdl/blob/1.0.6_3/package.js#L27 where I was trying to read the configs file from package.js. It worked when running locally but did not work at all once deployed.

[Zodiase]

#26 is where I realized I can't access the settings file from within package.js.

[ramijarrar]

Okay - makes sense. Have you considered removing the dependency altogether? I think the icons are sufficiently useful on their own, and having people install them separately isn't a huge burden (though obviously something that would be a breaking change).

[Zodiase]

Breaking BC has been the major concern for not removing the dependency. Also I would like most users to not have to remember installing multiple packages.

BTW I may have realized a better solution than forking and editing just for this purpose. Say if you have a local package that matches zodiase:material-design-icons-fonts but actually does nothing, wouldn't that just effectively removes what you don't need? And you also get to keep this package updated.