User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en-US; rv:1.0.1) Gecko/20021208 Chimera/0.6+ Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en-US; rv:1.0.1) Gecko/20021208 Chimera/0.6+ The example site I've given has a form for posting messages on the messageboard. It has a subject field, a message field and an email address and password field. When posting, Chimera recognises there is a password field on the forum and offers to store the password. But rather than store the email address and password it stores the subject and password. This means every time I goto post messages the subject line is replaced with the subject of the message I was replying to when the password was remembered. This seems to be because Chimera is remembering the first field in the form as the username rather than whatever field is before the password field. So if subject is 1, main text is 2, email is 3 and password is 4, Chimera should recognise, like IE does, that the username email is most likely to be field 3 (one field before password) than field 1. Reproducible: Always Steps to Reproduce: 1. Goto post new message or post reply. 2. Type in your message, email address and password. 3. Click submit and agree to remember password. 4. Go try post on another topic and see it fills in the subject line rather than email field. Actual Results: Subject field and password is remembered. Expected Results: Email address and password field should be remembered.
Assignee: bryner → pinkerton
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: when storing login details for autocompletion, the wrong username field is remembered → when storing login details in keychain, the wrong username field is remembered
yeah, we just remember the first field in the form.
Target Milestone: --- → Chimera1.1
Created attachment 147404 [details] [diff] [review] Better guess at username field Fixes username guessing to respect autocomplete="off" Uses the last autocompleteable text field before the password field as username, rather than the first text field.
This should probably go on the 0.8 branch as well as the trunk
Comment on attachment 147404 [details] [diff] [review] Better guess at username field I looked over, applied, and tested this patch thoroughly. It does indeed fix the problem at the URL mentioned (I signed up for an account on the forum and verified the fix there).
Comment on attachment 147404 [details] [diff] [review] Better guess at username field sr=pink
Attachment #147404 - Flags: superreview?(pinkerton) → superreview+
landed on trunk.
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.