-
Notifications
You must be signed in to change notification settings - Fork 115
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
Liga SF Mono fonts clobber the original SF Mono fonts #15
Comments
The renaming stuff is, honestly, a quick hack that I did not test rigorously and (in particular) assumes:
I'm not sure what's going on with SF Mono. I think this should be fixable by looking at the font metadata first and prepending "Liga " to it, and falling back to filename-based naming only if that fails. |
Thanks @ToxicFrog. I guess the exact names don't matter so much, but where on earth is that "Liga " prepend happening? I guess it has something to do with Ohhhh. If I'm reading it correctly, "Liga" (no space) is prepended to the output file name coming from the Makefile: https://github.com/rojiani/Ligaturizer/blob/828d376809fc83557a7db82949fb2911b059dfce/Makefile#L16 and the Python code above is using camel case to reverse engineer everything from the filename. The SF Mono case is a weird one though. It looks like all the metadata is the same as the original, almost like it was a straight copy. |
All the metadata is copied from the original, and then some specific fields are overriden in change_font_names based on the information inferred from the filename. The I haven't really had a chance to look into this yet, but it will probably involve dumping all the metadata from fonts like SF Mono and seeing where it differs from fonts that work as expected. |
This should be fixed for Edlo, IBM Plex Mono, and Meslo fonts as of 670d05e. It is not fixed for SF Mono (in either weight), and I have no idea why -- I can see the field getting set properly, but then the original field value gets written to the new font file. Further investigation required. |
When running make for SF Mono in the output there is: This is not true for the other fonts. So for some reason font merge refuses to rename SF Mono. |
This might have been fixed in 6d3070a. |
still is there, I think |
I just went to install all the input and output fonts, and when installing the output fonts, I found that I already had "SF Mono Regular" and "SF Mono Semibold" - it looks like these two are the only output fonts which have exactly the same name as their input fonts. Most of the fonts have "Liga " prepended.
However, a couple of the other fonts are named a little inconsistently, because they don't have a space after "Liga", ie
Also the Meslo LG fonts have different spacing in Liga form, eg "Meslo LG S DZ Regular" became "Liga MesloLGSD Z Regular".
The text was updated successfully, but these errors were encountered: