-
-
Notifications
You must be signed in to change notification settings - Fork 725
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Webfont Ligatures (for Angular Material, etc...) #1013
Comments
You could definitely use MDI in Angular Material with the regular class approach, but it sounds like you're asking if there is ligature support. Currently there is no ligatures ability provided (it's something that could be provided, no one's requested it yet, as I assume most just use classes for bootstrap). I'm not 100% sure I answered the question. |
More or less that's it :), so integration is the same as font awesome etc. Thanks |
Might be useful to provide ligature support in the future though. I'm going to rename this issue and mark it as a webfont feature. |
Any intent of doing this? This could vastly simplify the use of this font as a in place replacement of the Official Material icons font. |
I strongly agree with @mbellezi here, there are frameworks that use Official Material Icons by means of ligatures. Having ligatures for this font will make it no-brainer to swap official one providing that all basic icons from official one have counterpart with corresponding ligature. |
By the way, this is a very low priority feature as we move to suggesting inline SVG over the webfont in the future. This would be an alternative font and not be included in the default font. The build tools we use I believe support ligatures, so I'll need to look into the steps required to add this. |
Hello Templarian, |
I've just whipped up a super quick and dirty grunt-webfont task, and it seems that (assuming you're using @Templarian, if you don't have time to work on this but are willing/able to share your existing build process, I can probably find some time to help out. |
@pauln I spent a few hours the other day working on this with the node side, but 1/3 of the icons are produced with errors in the glyphs. The plan is to update the SVG data that is incompatible and release a solution without the Font Forge dependency. A good solution you could build for the community right now would be...
The current build I have runs on a legacy version of |
Delving deeper, there do seem to be a few issues with the generated ligature font:
The |
I noticed that |
I've submitted a pull request to grunt-webfont which updates it to
I suspect that this incompatible SVG data is the reason that the fonts generated by my build script are missing some glyphs. The missing glyphs cause some interesting side effects with ligatures - some of them result in multiple other glyphs being displayed instead, such as |
Any news? |
Another use case where ligature support will be useful is Sketch App. |
Still kind of a low priority for our team at the moment, but I am going to be rewriting our webfont build script. During this step we'll have the ability to enable ligatures. We will not be including these in the webfont build, but will try to provide them for designers. This rewrite is our effort to setup nightly builds of all of the packages available on NPM right now. Once again, you should not be using the webfont for the web. The http://dev.materialdesignicons.com/guide/webfont-alternatives |
We've decided against doing this because it inflates the size of the font substantially for a niche market. |
I forgot to post I actually did add ligature generation, but it won't work due to the size of our font file. The ligature table can only encode so many characters before it just truncates. So only 2k out of the 5.5k were working. So can't even add it if we wanted to. |
Documentation specifies svg way to integrate icons with Angular Materials.
Maybe it's worth specifying approach to do the same (Angular Materials) but for web font integration?
(If possible - didn't investigate :) ) If not possible maybe worth as feature request.
The text was updated successfully, but these errors were encountered: