Skip to content
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

yaldevijaffna: v1.020 added #670

Closed
wants to merge 1 commit into from
Closed

Conversation

m4rc1e
Copy link
Collaborator

@m4rc1e m4rc1e commented Feb 20, 2017

No description provided.

@enathu
Copy link

enathu commented Feb 20, 2017

It seems few glyphs slots are misplaced in this font. As placement of Tamil U and Uu compound glyphs of N, Nn, and Nnn are not correct. According to the current version, the glyphs are formed in this way.

yaldevijaffna-regular-01

I have hotfixed the font with the correction for the preview. :) Hotfixed ttf is here. HotFixed-YaldeviJaffna-Regular.zip

Cheers.

yaldevijaffna-regular-hotfixed

@m4rc1e
Copy link
Collaborator Author

m4rc1e commented Feb 20, 2017

@enathu cheers, I'll investigate this.
cc @pathumego

@m4rc1e
Copy link
Collaborator Author

m4rc1e commented Feb 20, 2017

@enathu Thank you very much. I can confirm your find.

screen shot 2017-02-20 at 15 26 00

@pathumego @davelab6 I think we should resolve this in the upstream repo. This is not a code point issue.

Keep this open for the time being

@pathumego
Copy link

@m4rc1e @enathu Thanks, investigating

@davelab6
Copy link
Member

I agree to hold this open for now :)

@pathumego
Copy link

Fixed in upstream --> https://github.com/mooniak/yaldevi-fonts/releases/tag/1.030 Thanks @enathu

@m4rc1e
Copy link
Collaborator Author

m4rc1e commented Jul 19, 2017

Thanks @pathumego, I'll close this pr and another for the updated version.

@m4rc1e m4rc1e closed this Jul 19, 2017
@m4rc1e m4rc1e deleted the yaldevijaffna branch July 19, 2017 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants