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

api.CanvasRenderingContext2D.isPointInPath - Wrong Safari support info #9328

Closed
ramhugger opened this issue Mar 4, 2021 · 1 comment
Closed
Labels
data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API

Comments

@ramhugger
Copy link
Contributor

ramhugger commented Mar 4, 2021

System info:

Browser Version OS OS Arch
Safari 14.0.3 MacOS Big Sur 11.2.2 ARM

The information was incorrect

MDN Support Table (incorrect)

The documentation page in question claims that Safari does not support the method CanvasRenderingContext2D.isPointInPath() with the following syntax:

ctx.isPointInPath(path, x, y [, fillRule]);

How I tested this

After I had a look at the compatibility table which is online right now, I thought it was pretty weird for a browser like Safari not being featuring such a simple method for such a widely used Web API.
I tried then to call isPointInPath() using the unsupported syntax.
I'm sure you already guessed that it worked like a charm.

After that test I went to read the official Safari documentation at developer.apple.com.
Their parameters list doesn't say anything about the second syntax of this method, but it can be found in the method declaration; like this:

Apple Developer documentation (should be right...?)

I don't know which one is supposed be the correct one, and I don't know why all the other tables I could find about this method in Safari do not mention clearly the second syntax, but what I know is that it works without any issue and that it seems to be fully supported.

Because of the reasons discussed above, I think that Safari's support status for

Path parameter

should be changed to Full support since v10.0.

MDN page report details
@queengooborg queengooborg added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Mar 5, 2021
@ddbeck
Copy link
Collaborator

ddbeck commented Mar 11, 2021

Fixed by #9427

@ddbeck ddbeck closed this as completed Mar 11, 2021
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

No branches or pull requests

3 participants