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

Mark navigator.canShare() experimental and nonstandard #6844

Conversation

sideshowbarker
Copy link
Collaborator

This change marks the navigator.canShare() method as experimental:true and standard_track:false, per w3c/web-share#134 and w3c/web-share#134 (comment) in particular. w3c/web-share@2bbde9c dropped it as a specified feature (along with the entire Web Share Level 2 spec.)

This change marks the navigator.canShare() method as experimental:true
and standard_track:false, per w3c/web-share#134
and w3c/web-share#134 (comment) in
particular. w3c/web-share@2bbde9c dropped it
as a specified feature (along with the entire Web Share Level 2 spec.)
@github-actions github-actions bot added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Oct 6, 2020
@sideshowbarker sideshowbarker force-pushed the sideshowbarker/navigator.canShare-experimental-nonstandard branch from 239dddb to 463176b Compare October 6, 2020 04:20
@saschanaz
Copy link
Contributor

I think nonstandard things are just nonstandard, they shouldn't be marked as experimental: #6812

@sideshowbarker
Copy link
Collaborator Author

I think nonstandard things are just nonstandard, they shouldn't be marked as experimental: #6812

I don’t really have a strong opinion about that either way. But as far as the MDN side of this goes, it seems like there’s value to web developers in knowing both whether something’s experimental or not, and also knowing whether something’s nonstandard. And the difference it’s just hypothetical; I believe we have cases of features that are nonstandard which aren’t experimental.

@ddbeck
Copy link
Collaborator

ddbeck commented Oct 13, 2020

Relatedly, some questions on experimental: #6905. I kinda wish we had originally named experimental something like "stable." Supposing my guideline gets some support, I guess the question here would be, as a Web developer, should I expect navigator.canShare() to change in the future?

@sideshowbarker
Copy link
Collaborator Author

Relatedly, some questions on experimental: #6905. I kinda wish we had originally named experimental something like "stable." Supposing my guideline gets some support, I guess the question here would be, as a Web developer, should I expect navigator.canShare() to change in the future?

I think the answer to that question is Yes — based on w3c/web-share#134 (comment), which says:

we don't have consensus on canShare() or the shape of that API

…and on w3c/web-share#108:

I wonder if we can simplify canShare() a bit... right now, canShare() conflates type checking behavior by intertwining the method call with share(). This is problematic because share() returns a promise, not TypeErrors.

…and from reading the rest of the discussion in that issue, it seems clear one possibility is that the canShare() API could end up being changed in way to address the problems surfaced in that issue. (Another possibility is that canShare() will be replaced by an API with a different name — but that’s a different matter.)

In other words the gist of it is that the canShare() API is not stable.

@ddbeck
Copy link
Collaborator

ddbeck commented Oct 15, 2020

OK, thank you, @sideshowbarker. I think this will end up being experimental no matter what we ultimately decide in #6905, so I'm going to approve and merge this.

@ddbeck ddbeck merged commit fee6670 into mdn:master Oct 15, 2020
@sideshowbarker sideshowbarker deleted the sideshowbarker/navigator.canShare-experimental-nonstandard branch January 21, 2021 02:43
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.

None yet

3 participants