-
Notifications
You must be signed in to change notification settings - Fork 55
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
Update Persistence permission section #128
Comments
This is probably wrong as "permission state" is only defined for a descriptor? (Not entirely sure how to phrase this instead though.) Minor thing to fix: s/ |
At the very bottom of https://w3c.github.io/permissions/#reading-current-states it says (kind of easy to miss):
So we could leave it as is, or change the text to be something like:
|
Ah okay, I guess that's okay then. Although it does seem a little weird in that permission state is also an algorithm. I'm not sure if it always was like that. |
So let's close this as "done-ish". (feel free to re-open if you disagree!) |
As part of w3c/permissions#263, we're making sure all the powerful features are properly hooked up in each of their specs (w3c/permissions#329 tracks persistent-storage in particular).
Storage looks pretty good, there's a few busted links (I guess because Permissions was recently ported from bikeshed to respec? 🤷 ). And we can hook into https://w3c.github.io/permissions/#dfn-getting-the-current-permission-state as well.
This issue tracks that work - I'll send in a PR.
The text was updated successfully, but these errors were encountered: