Open
Bug 1317163
Opened 9 years ago
Updated 3 years ago
UX/UI review of identity auth window
Categories
(WebExtensions :: General, enhancement, P5)
Tracking
(Not tracked)
NEW
People
(Reporter: mixedpuppy, Unassigned)
References
Details
(Whiteboard: triaged)
The identity api opens a window so addons can have users authorize with a service. Should this happen in a tab instead? What about authorizations from non-browser windows?
Comment 1•9 years ago
|
||
markus - wasn't sure who on UX to assign this to. please move to correct person if it's not you. :)
Assignee: nobody → mjaritz
Priority: -- → P2
Whiteboard: triaged
Comment 2•9 years ago
|
||
Shane, is this already built? How can I test the current implementation?
How does Chrome solve this? Do you know a Chrome extension which which I could experience their implementation?
Flags: needinfo?(mixedpuppy)
Comment 3•9 years ago
|
||
I found a related issue in chromium that might help us learn how we should implement:
https://bugs.chromium.org/p/chromium/issues/detail?id=466154
| Reporter | ||
Comment 4•9 years ago
|
||
You can find an example addon here:
https://github.com/mixedpuppy/webextensions-examples/tree/master/identity
It's waiting to be merged to the mdn repo. The README has setup instructions.
The API implemented is this: https://developer.chrome.com/apps/identity#method-launchWebAuthFlow
The issue you linked to above shows what I have seen in Chrome, on Chrome a popup dialog is used to display the login pages from the services. The sole difference is that I've left the url bar in place. I feel it is important to show the domain being logged into.
The preference put forth by Kris in bug 1305421 is to place the logins into a tab and only use a dialog if/when the front window is not a tabbed browser window. For simplicity sake I prefer to leave it in a popup window.
Flags: needinfo?(mixedpuppy)
Comment 5•9 years ago
|
||
(In reply to Shane Caraveo (:mixedpuppy) from comment #4)
> The issue you linked to above shows what I have seen in Chrome, on Chrome a
> popup dialog is used to display the login pages from the services. The sole
> difference is that I've left the url bar in place. I feel it is important
> to show the domain being logged into.
I agree. Showing the URL gives users the option to validate the URL before entering their credentials.
> The preference put forth by Kris in bug 1305421 is to place the logins into
> a tab and only use a dialog if/when the front window is not a tabbed browser
> window. For simplicity sake I prefer to leave it in a popup window.
Having it as a popup in every case sounds like a consistent first step. I assume the most likely place to trigger an identity auth window would be a button in the toolbar, or it's attached doorhanger. (both no tabbed browser windows)
We could do a follow up on making it a tab in some cases if we see that many extensions implement tabs that initiate the identity auth window.
| Reporter | ||
Comment 6•9 years ago
|
||
(In reply to Markus Jaritz [:designakt prev :maritz] (UX) from comment #5)
> Having it as a popup in every case sounds like a consistent first step. I
> assume the most likely place to trigger an identity auth window would be a
> button in the toolbar, or it's attached doorhanger. (both no tabbed browser
> windows)
> We could do a follow up on making it a tab in some cases if we see that many
> extensions implement tabs that initiate the identity auth window.
This can be initiated anywhere an addon can run code. Could be a devtool addon, a panel, button, background page, etc. However the interaction is pretty poor from a panel (as it would be with a tab) since the panel closes and cannot be programmatically reopened. It is implemented per documentation which implies that user interaction is not required, but highly recommended.
Updated•8 years ago
|
Severity: normal → enhancement
Priority: P2 → P5
Comment 7•8 years ago
|
||
un-assigning my self as it looks it has become less of a priority, and I really haven't gotten around focusing on it.
Assignee: mjaritz → nobody
Updated•8 years ago
|
Product: Toolkit → WebExtensions
Comment 8•8 years ago
|
||
Bulk move of bugs per https://bugzilla.mozilla.org/show_bug.cgi?id=1483958
Component: Untriaged → General
Comment 9•7 years ago
|
||
Via feedback from not-yet-publicly-released project that I'm not going to link to from this public bug:
[Greg Tatum 9:56 AM]: Feedback: The sign in opened in a new window with limited browser chrome. I usually sign in through the lastpass menu item, and it's not present. I would prefer having it as a normal tab.
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•