We need to add some inline messaging when a new master password textboxes are empty. User can currently hit OK with blank entries, and nothing happens. Repro: 1) fennec nightly, 12-28-2011 build, SGS2 2) launch prefs, check the Use master password button 3) When the dialog box appears with the password / confirm password textboxes, leave it blank and hit OK. Alternately, skipping the password box, and just typing in confirmation does the same 4) verify no inline messages or action required to the user. Expected: - an inline message telling the user to fill out both boxes. also disabling OK if there's no entries. Actual; - no messaging
Changing to "Don't allow user to tap 'OK' if the password fields are blank"
Created attachment 586520 [details] [diff] [review] Patch
Comment on attachment 586520 [details] [diff] [review] Patch Make sure you test this by adding, removing and re-adding a master password. Just to make sure the dialogs work in cyclical usage.
Comment on attachment 586520 [details] [diff] [review] Patch [Approval Request Comment] Regression caused by (bug #): landing of mp on trunk User impact if declined: Very low. Users will be able to attempt to use an empty mp. it will fail Testing completed (on m-c, etc.): on m-c for a bit now Risk to taking this patch (and alternatives if risky): very low risk
Comment on attachment 586520 [details] [diff] [review] Patch [Triage Comment] Mobile only - approved for Aurora.
Verified fixed on: Nightly Fennec 13.0a1 (2012-03-06) Device: HTC Desire Z OS: Android 2.3.3 The OK button is not active until the two fields: password and confirm password are not identical and different of null.