Closed Bug 564523 Opened 15 years ago Closed 15 years ago

passphrase can match password for new accounts

Categories

(Firefox :: Sync, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: mconnor, Assigned: mconnor)

References

Details

Attachments

(1 file)

Attached patch kind of a hackSplinter Review
fallout from unification. this is kind of a hacky solution, I'll think about a more elegant solution later.
Flags: blocking-weave1.3+
Attachment #444182 - Flags: review?
Attachment #444182 - Attachment is patch: true
Attachment #444182 - Attachment mime type: application/octet-stream → text/plain
Attachment #444182 - Flags: review? → review?(edilee)
Comment on attachment 444182 [details] [diff] [review] kind of a hack >+++ b/source/chrome/locale/en-US/errors.properties > change.passphrase.ppSameAsPassphrase = The secret phrase cannot be the same as your current secret phrase > change.passphrase.ppSameAsPassword = The secret phrase cannot be the same as your password >+change.passphrase.ppSameAsUsername = The secret phrase cannot be the same as your user name >+change.password.pwSameAsPassphrase = Password can't match secret phrase >+change.password.pwSameAsPassword = Password can't match current password >+change.password.pwSameAsUsername = Password can't match your user name Any reason why we have different wording for passphrase vs password matching/being same?
Attachment #444182 - Flags: review?(edilee) → review+
There's more room in the secret phrase UIs for longer strings. The password fields are in grids which eat a lot of space, so we'd get ugly UI with longer strings. I'm probably going to file a bug on doing feedback using an XBL binding so we can play with using description or something similar to allow wrapping, without putting a lot of intelligence into callers. Also means language with longer strings aren't as screwed.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Flags: in-testsuite?
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: