-
-
Notifications
You must be signed in to change notification settings - Fork 639
Add glyphs U+012C..U+012D #327
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
Add glyphs U+012C..U+012D #327
Conversation
mynetx
commented
Sep 29, 2017
@chrissimpkins I did not touch the metadata – please advise if you’re going to do that on your own. |
No problem JM, I can definitely take care of the metadata. Thanks for adding these glyphs! Associated issue report for missing glyphs in the Latin Ext A set: #324 |
@jublo out of interest, did you modify the design in a different directory or add the plist files to .gitignore? |
@chrissimpkins I added the glyphs like always, and from the list of changed files, I discarded anything except those new files. |
df6587a
to
90d910d
Compare
Fixed horizontal position of the breve on lowercase letter in Regular now. |
90d910d
to
6e0572c
Compare
Renamed files to their correct names. |
Like this?
Or did you manually revert the plist files that editor changed? Trying to come up with a PR workflow that is simple and does not modify the plist XML files |
so you just work in a dirty repository with the plist modifications that happen with the edits and do not commit those? |
@chrissimpkins That's right. |
merged into |
tests passed. merged to |