|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
59 bytes, text/x-review-board-request
|Details | Review|
Affected versions]: Tested on Nightly 59.0a1(2018-01-04) [Affected platforms]: Tested on Mac OS X 10.12, Windows 10 [Steps to reproduce]: Prerequisites: 1. Make sure you have at least two saved profile. STR: 1. Go to https://www.canadiantire.ca and buy an item 2. In the billing form double click on "First Name" or "Last Name" [Expected result]: The autofill suggestions are displayed. [Actual result]: Nothing happens. NOTE: Here is a video of the actual result: https://streamable.com/uha48 We checked after autocomplete=off but we didn't see this attribute for "First name" or "Last name".
I'll look into it soon.
Assignee: nobody → lchang
Whiteboard: [form autofill: V2] → [form autofill:V2][Misc.]
I just looked into it and it seems the heuristic problem that first name/ last name/address line 2/ city/ postal code/ tel fields will be detected as tel-extension, only address level 1 (province)/ country/ email/ tel-extension fields are correct. This also causes the unnecessary sectioning because of duplicate tel-extension fields, so the rest of the correct fields could not be filled because there's no sufficient valid fields in a section.
This issue is caused by missing detection tel-extension by its regexp rule (`"\\bext|ext\\b|extension"`), and Bug 1392932 is a similar one. I submit a WIP patch to determine tel-extension from the regexp list to telephone parser only, and there are some test fails have to be resolved.
Assignee: lchang → selee
Status: NEW → ASSIGNED
This patch is based on the solution I purposed in comment 4, and it fixed the issue in bug 1392932. In addition, this patch will be rebased on bug 1417834 later since bug 1417834 is not landed yet.
Comment on attachment 8940628 [details] Bug 1428292 - Determine tel-extension in telephone parser only. https://reviewboard.mozilla.org/r/210860/#review218076 Looks good.
Attachment #8940628 - Flags: review?(lchang) → review+
Thanks for the review. The patch is rebased to the latest m-c.
Pushed by firstname.lastname@example.org https://hg.mozilla.org/integration/autoland/rev/860a997f95c9f7961efd94b48ffb6500afb3b1a9 Determine tel-extension in telephone parser only. r=lchang
Status: ASSIGNED → RESOLVED
Last Resolved: 9 days ago
status-firefox59: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Hi Ovidiu, can you help us to verify these bug? Thanks.
I verified this on Mac OS X 10.12, Windows 10 x64 and Ubuntu 16.04 with Nightly 59.0a1(2018-01-14) and I can confirm the fix, I will mark this as verified fixed.
Status: RESOLVED → VERIFIED
status-firefox59: fixed → verified
You need to log in before you can comment on or make changes to this bug.