-
Notifications
You must be signed in to change notification settings - Fork 16
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
Asking for permission #36
Comments
Given that A permission request is best done asynchronously.
What are your thoughts about this? Alternatively, exploring an explicit permission request, what do you think of a This is in the absence of The reasoning to make this an explicit request would decouple the async iterator request with a permission requests and therefore makes a worker implementation more palatable. It seems developer-hostile to get developers to call |
I'd rather prefer |
It depends, one thing that wasn't clear to me is whether an async iterator is warranted (see #32). Are there concrete plans to expose this in workers? |
RE: the async iterator, I'll give it some thought and respond on #32 RE: exposing to workers, there aren't concrete plans as of yet. However, the developers we've been obtaining feedback from want to use the data obtain from the API in workers. Do you see concerns exposing this in workers? |
The potential problem I see is with the UI flow. I think ideally the user is empowered to chose what fonts end up exposed. And although that remains possible if you separate the permission prompt from the query, it all becomes a bit indirect. |
With the font dialog picker things may get more complicated when the tool is collaborative. A designer may select an initial set of fonts and the others should have a way to accept that list. During design the other collaborators (or even him) may want to change that initial set. A new font added into the system should generate user activation/permission requests for all the users involved. I feel that such amount of approvals that need to be given would make the application unattractive. |
The API was change d so that permission is requested during https://wicg.github.io/local-font-access/#dom-fontmanager-query |
Thanks, that's great! |
I think it would be better if the permission request is tied to
query()
. The end user flow I envision is that the website invokesquery()
and the user then gets a browser-driven dialog with which they can select zero or more fonts to expose to the website.(Aside: Mozilla is not interested in
navigator.permissions.request()
and it's not even part of Permissions.)The text was updated successfully, but these errors were encountered: