(In reply to Betsy Mikel [:betsymi] from comment #6)
Yes, let's get Johann's perspective for sure!
Temporary usually means for the lifetime of that particular tab / until the user refreshes the page. I think that's a concept many of our users understand.
I am not confident we can say that. I would say that is an assumption.
While I'm also not sure this is clear to users you have to admit that we're all arguing based on assumptions here, if we wanted to make data-driven decisions then this bug would be stalled on collecting more data. :)
In addition, this little UI is quite dense and this extra word in the label is not doing us any favors.
I agree that the extra text is crowding the panel.
If you have several permissions as once, the text wraps in strange ways
To be more accurate, it's independent of the amount of permissions and, in English, only happens for the persistent storage permission because that also has a long permission name. So yes, it could potentially happen in other languages. Whether that wrapping is strange or unexpected is up for debate, I guess.
and begins to expand, covering the page content.
What? I'm not sure what you mean by that. The text is going outside of the panel?
So, overall I can see both sides here. On one hand, I think it would be nice to have a cleaner layout for this dialog with more space and less information for the user. On the other, I agree it is pretty important to make the distinction between a permanent and temporary permission visible, especially since it serves as feedback to the actions the user took in the permission dialog (and a few more advanced users will also expect this). This is especially important for Camera/Mic access, where this indicator serves two different purposes:
- While recording, the user has no other means of ensuring that the given camera access is actually one-off, not permanent. If they wanted to remove the permission to be sure, they'd stop the recording as a side effect. We should not motivate that.
- The temporary compatibility grants that Paul mentioned are not associated with a user action. By stating that that grant is temporary we can hopefully hint to the user that this isn't something to worry about.
Nonetheless, I totally see your point about specifically the word "temporarily" being too ambiguous.
My suggestion would be to change the text to something like "Allowed for this Tab" to help reduce confusion and slightly widening the panel to help with the layout issues. I don't think we can perfectly solve one unless we're willing to stop caring about the other, but maybe this brings us closer to a good compromise.