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

Fix isPointInPath Path Parameter support info for Safari #9427

Merged
merged 2 commits into from
Mar 11, 2021
Merged

Fix isPointInPath Path Parameter support info for Safari #9427

merged 2 commits into from
Mar 11, 2021

Conversation

ramhugger
Copy link
Contributor

@ramhugger ramhugger commented Mar 11, 2021

This Pull Request is the continuation of and closes #9353.
I changed Safari isPointInPath Path Parameter support info from false to "7", as suggested by @foolip in the original Pull Request #9353.

Change Safari isPointInPath Path Parameter support info from false to "7", as suggested by @foolip in #9353.
@github-actions github-actions bot added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Mar 11, 2021
api/CanvasRenderingContext2D.json Outdated Show resolved Hide resolved
Co-authored-by: Daniel D. Beck <daniel@ddbeck.com>
@ddbeck ddbeck changed the title Fix Safari support info - Part 2 Fix isPointInPath Path Parameter support info for Safari Mar 11, 2021
Copy link
Collaborator

@ddbeck ddbeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! 🎉

@ramhugger
Copy link
Contributor Author

It's me to thank you @foolip and @ddbeck 🥳!

@foolip
Copy link
Collaborator

foolip commented Mar 11, 2021

@ramhugger your change prompted me to take a closer look at some of the other entries, and I ended up changing a bunch more to be "7" in #9433.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants