Permissions query() should not spawn UI

WICG / local-font-access

Web API for enumerating fonts on the local system

https://wicg.github.io/local-font-access

Apache License 2.0

77 stars 16 forks source link

Closed annevk closed 2 years ago

annevk commented 2 years ago

Unless I'm mistaken, https://wicg.github.io/local-font-access/#permissions seems somewhat bogus. query() isn't there to obtain a permission, it just gives you the current state. It shouldn't trigger any UI.

cc @marcoscaceres @miketaylr

inexorabletash commented 2 years ago

Ugh, that section of the proposal is just busted as a result of a bad find/replace.

"When the query() API is invoked, the user agent may present a list of font choices, a yes/no choice, or other interface options." - that should be referring to the proposed queryLocalFonts() API.
The example was updated to reference the queryLocalFonts() API, but it's supposed to be referencing navigator.permissions.query(), i.e. testing the state of the permission. I'll fix.