Material icons from npm

[danilopolani]

Using npm module of material icons there is no longer a need to include the icons with an import/link, so you can use the datatable also in offline projects.

PS: I used @import('~material..') but I don't know if it's compatible for plain CSS/Webpack and other

[thepill]

@MicroDroid I´m not sure about that - this will increase the size and it could be possible that the users already fetching the font from google, so they will download unnecessary data. Also i'm not sure about the @import statement.

@danilopolani For upcoming ideas it could be great if you first could create a issue to tell your idea. We could argue about that and It prevents you from implementing something wich would not necessarily be merged into it. Thanks :)

[MicroDroid]

I'm just thinking, given that the user is already loading Material Icons from some other source, wouldn't this forcefully load it again since it's seemingly coming from a new source? and possibly end up with the font being loaded more than once in the final bundle?

Me thinks it's better to keep the font external so we're sure its loaded once plus loading it from a CDN makes the browser benefit from caching, as it's very likely for the browser to already have a cached copy of such resource. Not entirely sure though myself about it.

[danilopolani]

Okay, then I think you can just add the npm module to the Readme as alternative to google web fonts, can I create a PR with the suggestion on Readme?

[MicroDroid]

I think yes, as an alternative to Google Web Fonts.

[MicroDroid]

Wait we can keep this open 🤔