Skip to content

Add Path2D string constructor #277

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

Closed
wants to merge 2 commits into from
Closed

Conversation

thw0rted
Copy link

@thw0rted thw0rted commented Aug 3, 2017

Per microsoft/TypeScript#14660, Path2D should have a constructor overload that takes a single `string. This must be added here to be consistent with the WhatWG Canvas spec, which is a Living Standard.

@thw0rted
Copy link
Author

thw0rted commented Aug 3, 2017

I'm passing the CLA to my management, hopefully it's something they can clear quickly. Sorry for the wait.

@mhegazy
Copy link
Contributor

mhegazy commented Aug 15, 2017

@thw0rted we will not be able to proceed with this PR without the CLA.

@thw0rted
Copy link
Author

I will bug my management about this again, I had no idea they would take so long getting a response to me. Sorry!

@mhegazy
Copy link
Contributor

mhegazy commented Aug 15, 2017

I will bug my management about this again, I had no idea they would take so long getting a response to me. Sorry!

just a reminder you do not need your employers sign off if you do this on your own time.

@microsoft microsoft deleted a comment from msftclas Sep 26, 2017
mhegazy added a commit that referenced this pull request Mar 8, 2018
@mhegazy mhegazy mentioned this pull request Mar 8, 2018
@mhegazy mhegazy closed this in #387 Mar 8, 2018
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