Improve copy for disambiguating login update

RESOLVED FIXED in Firefox 55

Status

()

P3
enhancement
RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: rfeeley, Assigned: squib)

Tracking

unspecified
mozilla55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

(Whiteboard: [passwords:capture-UI])

Attachments

(3 attachments)

(Reporter)

Description

3 years ago
When users change their password on a website, sometimes the username is not detectable, and if the user has more than one login for a site, they can select the login they would like to update.

Currently we present (screenshot attached)

* Please confirm which user you are changing the password for
* [list of usernames]
* [ OK - primary ] [ Cancel ]

I think the copy could be improved to something like:

* Select the login you are updating:
* [list of usernames]
* [ Update - primary ] [ Cancel ]

But we are open to suggestion…
(In reply to Ryan Feeley [:rfeeley] from comment #0)
> Created attachment 8683694 [details]
> Screenshot 2015-11-03 13.47.51.png
> 
> When users change their password on a website, sometimes the username is not
> detectable, and if the user has more than one login for a site, they can
> select the login they would like to update.
> 
> Currently we present (screenshot attached)
> 
> * Please confirm which user you are changing the password for
> * [list of usernames]
> * [ OK - primary ] [ Cancel ]
> 
> I think the copy could be improved to something like:
> 
> * Select the login you are updating:
> * [list of usernames]
> * [ Update - primary ] [ Cancel ]
> 
> But we are open to suggestion…

Looks good, though I think we can shorten it further:

* Select which login to update:

Or a little longer, but friendlier:

* Choose which login you'd like to update:

I'm going back and forth between "login" and "user," though. Will "login" be clear to everyone? I always worry it sounds like jargon.
(Reporter)

Comment 2

3 years ago
Thanks to Gemma, we have some research from February that indicates users are very comfortable with the term "login" and seem to prefer it in this context (a combination of username and password).

Her deck:
https://docs.google.com/presentation/d/17iw3ls5AN2FbbSqVdV_i_-PHp6tXbmmw5-xdA4vS5ak/edit#slide=id.g77b514185_0113

"Select which login to update:" is perfect. Easy to take in at a glance and make a quick decision, describes action accurately.

Thanks!
(In reply to Ryan Feeley [:rfeeley] from comment #2)
> Thanks to Gemma, we have some research from February that indicates users
> are very comfortable with the term "login" and seem to prefer it in this
> context (a combination of username and password).

Awesome, thanks!
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Reporter)

Comment 4

3 years ago
Posted image select-login.png
One for the backlog of simple login manager changes mattn? I think I will see if bwinton's qx posse might want to tackle this tiny change.

* Select which login to update:
* [list of usernames]
* [ Update - primary ] [ Cancel ]
Flags: needinfo?(MattN+bmo)
(Reporter)

Updated

3 years ago
Assignee: matej → MattN+bmo
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Reporter)

Updated

3 years ago
Severity: normal → enhancement
Priority: -- → P3
Whiteboard: [passwords:capture-UI]
LGTM
Status: REOPENED → NEW
Flags: needinfo?(MattN+bmo)
(Assignee)

Updated

2 years ago
Assignee: nobody → squibblyflabbetydoo
Status: NEW → ASSIGNED
(Assignee)

Comment 6

2 years ago
There's an identical string for mobile too. Should I just update the string in toolkit or should I change mobile as well? Here are the DXR results if they help: <https://dxr.mozilla.org/mozilla-central/search?q=userSelectText&redirect=false>.
Flags: needinfo?(rfeeley)
(Reporter)

Comment 7

2 years ago
Great idea! I didn't know mobile had this.
Flags: needinfo?(rfeeley)
Comment hidden (mozreview-request)
(Assignee)

Updated

2 years ago
Attachment #8848305 - Flags: review?(MattN+bmo)
(Assignee)

Comment 9

2 years ago
This patch contains both the desktop and mobile changes. If it makes sense for someone from the mobile team to look at this, just let me know!
(Assignee)

Comment 10

2 years ago
Comment on attachment 8848305 [details]
Bug 1222032 - Improve copy for disambiguating login update

Clearing review; I just realized I forgot to change the "Ok" string to "Update".
Attachment #8848305 - Flags: review?(MattN+bmo)
(Assignee)

Comment 11

2 years ago
Comment on attachment 8848305 [details]
Bug 1222032 - Improve copy for disambiguating login update

Nevermind, re-requesting review. It looks like replacing "OK" with "Update" would be pretty tough (I'd either have to reimplement my own version of nsIPromptService.select or enhance that function to accept button labels).

I can do that if you want (the latter seems like the better way), but it'd probably be best to do it in a separate bug. Maybe just as a followup to this one? I'm a big fan of descriptive button labels, so I can definitely see some value in improving nsIPromptService.
Attachment #8848305 - Flags: review?(MattN+bmo)
Comment hidden (mozreview-request)
(Assignee)

Comment 13

2 years ago
Also: are there any example sites I can try this out on? I want to make 100% sure that this actually works right, but I don't know of any sites that would trigger this.
(Reporter)

Comment 14

2 years ago
It's commonly triggered when you change your password as the email is not present there. Using a site you have a real account with, log in with a fake login and save it. And then change your password. Should trigger it.
Comment on attachment 8848305 [details]
Bug 1222032 - Improve copy for disambiguating login update

https://reviewboard.mozilla.org/r/121228/#review126012

(In reply to Jim Porter (:squib) from comment #11)
> It looks like replacing "OK" with "Update"
> would be pretty tough (I'd either have to reimplement my own version of
> nsIPromptService.select or enhance that function to accept button labels).

Yeah, missing this change was the reason I didn't rubberstamp it when I first received the review… I went down the path of investigating how much work it would be since I didn't realize that the `select` API didn't support specify the button labels like one of the adjacent APIs does when I suggested this bug for you.

> I can do that if you want (the latter seems like the better way), but it'd
> probably be best to do it in a separate bug. Maybe just as a followup to
> this one? I'm a big fan of descriptive button labels, so I can definitely
> see some value in improving nsIPromptService.

Yeah, please file it as a follow-up. This is still an improvement as-is.
Attachment #8848305 - Flags: review?(MattN+bmo) → review+
(Assignee)

Comment 16

2 years ago
Filed bug 1353154 for the nsIPromptService.select issue.

Comment 17

2 years ago
Pushed by squibblyflabbetydoo@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/a42e01bc5c42
Improve copy for disambiguating login update r=MattN

Comment 18

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a42e01bc5c42
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago2 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.