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

get more font's data via psd-enginedata #72

Merged
merged 5 commits into from
Jul 14, 2017
Merged

get more font's data via psd-enginedata #72

merged 5 commits into from
Jul 14, 2017

Conversation

leenxyz
Copy link
Contributor

@leenxyz leenxyz commented Oct 11, 2016

psd.js miss some font's data,I think we should get more

@fabulousduck fabulousduck self-requested a review July 10, 2017 14:50
@fabulousduck
Copy link
Collaborator

@forsigner Thanks for the PR. Have you tested this ? Can you confirm this works ?

@fabulousduck
Copy link
Collaborator

fabulousduck commented Jul 11, 2017

@meltingice after having tested this briefly this PR does not seem to break any other functionality and can be merged. i would however advice you look through this yourself to see if you can spot anything.

This PR itself adds quite a lot of helpful data, so if it looks good to you, this can definitely be merged

This PR would also solve the issue brought up in #70.

@leenxyz
Copy link
Contributor Author

leenxyz commented Jul 13, 2017

@fabulousduck thx~

@leenxyz
Copy link
Contributor Author

leenxyz commented Jul 13, 2017

This repos still be maintained... Orz

@fabulousduck
Copy link
Collaborator

@forsigner this repo is being mainained as of late, the owner @meltingice is busy at work on a vanilla js rework of the entire project, meanwhile, i keep tabs on issues, and PR's people make.

I can't give any time frame when the vanilla rework will be done as this is completely up to @meltingice.

@meltingice meltingice merged commit 8a9a875 into meltingice:master Jul 14, 2017
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.

3 participants