Closed Bug 601019 Opened 14 years ago Closed 14 years ago

Sync UI: Always show "I have lost my Sync Key" and "Reset Password"

Categories

(Firefox :: Sync, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
blocking2.0 --- betaN+

People

(Reporter: philikon, Assigned: philikon)

References

Details

(Whiteboard: [strings][has review][has patch][softblocker])

Attachments

(2 files, 2 obsolete files)

If you go through the "I already have a Sync account" part of the setup wizard and don't remember your passphrase, you first have to enter something into the text field, click Continue/Next and *then* you see the "I have lost my Sync Key" option. We should really always show that as an option.
Possibly also always show "Reset Password" (same issue, different item).
Summary: Always show "I have lost my Sync Key" → Always show "I have lost my Sync Key" and "Reset Password"
Talked with faaborg about this. "I have lost my Sync Key" should be moved below the paragraph ending in "Show me how." We should also add a sentence before it a la "If you no longer have access to your other computer, you can reset your Sync Key." Nominating this for b9 blocking.
blocking2.0: --- → ?
Whiteboard: [strings]
Assignee: nobody → philipp
Attached patch v1 (obsolete) (deleted) — Splinter Review
Attachment #498540 - Flags: review?(mconnor)
Attached image screenshot of v1 (obsolete) (deleted) —
Attachment #498541 - Flags: feedback?(faaborg)
Can the reviewers do a grammar review on the strings, too? We should get them "comma perfect" and stuff the first time.
Summary: Always show "I have lost my Sync Key" and "Reset Password" → Sync UI: Always show "I have lost my Sync Key" and "Reset Password"
Attachment #498540 - Flags: review?(mconnor) → review+
My review is that this is too wordy. At least 50% fewer words, please.
blocking2.0: ? → betaN+
Whiteboard: [strings] → [strings][has patch][has review][needs feedback faaborg]
Whiteboard: [strings][has patch][has review][needs feedback faaborg] → [strings][has patch][has review][needs feedback faaborg][softblocker]
Comment on attachment 498541 [details] screenshot of v1 I'm worried about users claiming they have lost it (when they haven't really lost it) in hope that we will recover it and email it to them. Or because they don't know what it is so they just figure it is easier for us to fire off an email. What they are hoping for is a shortcut, and what they are getting is the opposite of a short cut (a long cut?), since we'll fire errors on the other machine, and just make them move the key either way. The second sentence is currently: "You can reset your Sync Key if you lost it and no longer have access to your other device. _I have lost my Sync Key_" I think we should change this to just: "I have lost my other device" So this makes the context of the question "do you have your other computer?" (yes) instead of "do you have your [unintelligible]?" (no?) So the full text: You can get a copy of your Sync Key by going to Sync Options on your other device, and selecting "My Sync Key" under "Manage Account". Show me how. I have lost my other device.
Attachment #498541 - Flags: feedback?(faaborg) → feedback-
Whiteboard: [strings][has patch][has review][needs feedback faaborg][softblocker] → [strings][has patch][has review][softblocker]
Thanks, Alex. That's clear and concise and simplifies the text.
Whiteboard: [strings][has patch][has review][softblocker] → [strings][has review][needs revised patch][softblocker]
Can this land today? I really don't want strings to land past this week.
(In reply to comment #12) > Can this land today? I really don't want strings to land past this week. Yes, the revised patch will be simple to do.
Attached patch v1.1 (deleted) — Splinter Review
Incorporate faaborg's feedback.
Attachment #498540 - Attachment is obsolete: true
Attached image screenshot of v1.1 (deleted) —
Attachment #498541 - Attachment is obsolete: true
This is good to go now!
Whiteboard: [strings][has review][needs revised patch][softblocker] → [strings][has review][has patch][softblocker]
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Comment on attachment 503876 [details] [diff] [review] v1.1 > <row align="center"> >+ <spacer/> >+ <label class="text-link" >+ value="&resetPassword.label;" >+ onclick="gSyncUtils.resetPassword(); return false;"/> >+ </row> You added a tab here. Please fix.
(In reply to comment #18) > You added a tab here. Please fix. Thanks for pointing that out. Fixing this over in bug 626099.
No longer blocks: 615950
Component: Firefox Sync: UI → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: