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

Added UMD, rename clearPath back to clear, fix clear method... #26

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

redexp
Copy link

@redexp redexp commented Feb 12, 2020

Fixes without breaking changes. Added UMD, rename clearPath back to clear, fix clear method, added .d.ts, fix Gruntfile, removed spec folder (using svg.js as dev deps), fix readme.

Sergii.Kliuchnyk added 2 commits February 12, 2020 18:37
….ts, fix Gruntfile, removed spec folder (using svg.js as dev deps), fix readme
@ghost
Copy link

ghost commented Feb 12, 2020

SVG dot js lib cutted in your merge request.
Recently I have been thinking about trying to use an SVG image to make animated drawing some Chinese-like characters on the canvas based on the meaning of the sentence passed to the input of module.

Doing review of your request with updated dude i am wondering why did you change your mind and decided to remove SVG dot js lib dependency from latest requested update of repository, explain your point of view.

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.

1 participant