-
Notifications
You must be signed in to change notification settings - Fork 10
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
"User notifications" section should be more specific #194
Comments
I believe what goes beyond "show some form of unobtrusive notification" would be an implementation detail due to diversity of browser and OS UI/UX, device form factors and screen sizes etc. @pes10k do you have an example of an exemplary web spec that goes into further detail we could learn from? |
I dont have a precise example off hand, but you might consider some of the WebPayments and WebAuthn work, which details in places what kinds of elements should be presented by the browser, and what text should be present, etc. The bigger point is that, if the group thinks that a user notification is an important privacy mitigation, I think its important to say more about the properties that notification should have. Presumably a |
Thanks @pes10k. I looked at those two specifications you mentioned. Do you perhaps refer to the following text as good examples to look for inspiration:
Note in https://www.w3.org/TR/payment-request/#show-method
Step 6 of https://www.w3.org/TR/webauthn-2/#sctn-sample-authentication
3rd text block in https://www.w3.org/TR/webauthn-2/#sctn-api Please let us know if this level of abstraction would be helpful for this spec too to guide implementers on user interface aspects in respect to the user notification. As you know, there's a limit to how much we can normatively specify in terms of UI/UX, but we agree this section would benefit from updates. After we hear back from you we'll enhance the user notification section of the specification and seek your review for the proposed improvements. Thank you! |
Yes, i think these are good models to build from. Thanks! |
While pondering this I realized that a change in one important keyword in this section could make a major difference in how implementers will perceive and approach this recommendation. @pes10k, what do you think if we'd change "unobtrusive" to "user-visible"? I personally prefer concise guidance because it is more likely to be read (and adhered to) by implementers. Here's how this section would read with this improvement (emphasis mine):
AFAICT the adjective "unobtrusive" does not have a special meaning in web specs and its dictionary definition reads:
"Unobtrusive" could indeed be interpreted so that a console.log message could suffice which clearly is not an appropriate way to inform users. OTOH I see "user-visible" appear in a few web specs. While also not a web spec defined concept, it is more concrete and better understood by non-native speakers I believe. If there's a better word for this please let us know! @pes10k maybe this is the significant change we need in this section for now? Small yet impactful. Thank you for your continued contributions! (As a bonus, this updated user notifications section could in part also help mitigate the proposed cross-site covert channels attack discussed in #197. Perhaps we could note in the mitigations section that if an implementation detects a long-running script that is using this API we could recommend this user notification to become more prominent akin to unresponsive script warning in some browsers.) |
@pes10k we believe #218 addresses this issue. Feel free to propose any additional text for inclusion into this User notifications section as appropriate. We will revisit this section if new information emerges or if there are further learnings or user feedback from the ongoing trial. Thanks for your suggestions! |
This issue is being filed as part of the requested PING review
Theres a "user notifications" recommendation section (which is good), but I think more detail is needed for this section to be helpful to implementors. For example, whats the scope of the notification (the entire browser? What happens if a page uses it in the background, etc?). How do the APIs react if a user uses the notification dialog to revoke access in the middle of the document executing? etc.
The text was updated successfully, but these errors were encountered: