-
Notifications
You must be signed in to change notification settings - Fork 53
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
Move permissions out to each spec #263
Comments
I tried to do this at some point by having each spec |
My plan is to separate the concept of a "permission" (a new definition type), and those independent of the enum values. Specs now routinely define an "Integration with Permissions Policy" section where they identify the permission string they are using. Our spec will just define the enum and values, but not claim to be the permission itself. That way, we can avoid the auto-linking issues (🤞) and everyone can own their own stuff. |
@marcoscaceres Somewhat related to this, should It isn't totally clear what the mapping "should" be (to me). Is the following correct?
Further, more closely related, if you move the permissions out to specs it would be really helpful to provide some information about where they are going to. Then we can update the browser compatibility info https://developer.mozilla.org/en-US/docs/Web/API/Permissions_API#browser_compatibility with spec links. |
No, and I totally understand that it looks like there is a 1:1 mapping. This is something I'm actually trying to fix. This spec will only deal with APIs that prompt for UI: like Geolocation. I added some text to try to explain the relationship between the two specs: For the case of "web-share", there is no permission prompt UI in any browser - hence, it's not part of this spec.
Yes, what we call "power features" in our spec. Powerful features (e.g., geolocation) need to get "permission" via some UI.
Correct (I wish Permissions Policy wouldn't of changed their name - it's made everything very confusing!)... A "permissions" per the Permission Policy spec control the availability of an API in a given context, for example: <!-- this allows geolocation -->
<iframe allow="geolocation" src="https://third-party.com"></iframe>
Sure! should I ping you directly? I can add something to our pull request template. |
Thanks again for your help. |
Where applicable, we should add a "Permission prompt" heading/section to specs and add:
|
Remove references to enum PermissionName and refer to powerful feature 'name' dfn instead. Helps with w3c/permissions#263
The Permissions registry seems a roundabout way to maintain things.... I think we definitely keep the various parts other specs need to define (plus an example of what other specs need to do), keep the enum, but then move all permission descriptions to their respective specs.
That allows this spec to "gate-keep" the main enum for the API, but allows other specs to define their own permissions.
TODO:
The text was updated successfully, but these errors were encountered: