Closed Bug 1645735 Opened 4 years ago Closed 4 years ago

[Staples.com] Exp date format is incorrectly auto-filled and highlighter overlaps CC logo placeholder

Categories

(Toolkit :: Form Autofill, defect, P2)

Desktop
All
defect

Tracking

()

VERIFIED FIXED
mozilla80
Tracking Status
firefox77 --- disabled
firefox78 --- disabled
firefox79 --- disabled
firefox80 --- verified

People

(Reporter: tbabos, Assigned: abr)

References

(Blocks 2 open bugs, )

Details

(Whiteboard: [cc-autofill-mvp])

Attachments

(3 files)

Attached video Recording of the issue

Affected version:
All - if CC specific prefs are enabled

Tested on:
Windows 10
MacOS 10.13
Ubuntu 16.04

Steps to reproduce:

  1. Reach the payment form on staples.com
  2. Click on the Exp Date field and select any entry from the atuofill dropdown (notice preview and date format in dropdown)

Expected results:

  1. The expiry date is correctly formatted to ensure field validation pass.
  2. Card number autofill highlight does not overlap the card logo placeholder for the field.

Actual results:

  1. The expiry date is incorrectly formatted (also cut off) and does not pass field validation
  2. Card number autofill highlight overlaps the card logo placeholder for the field.

Note:
Exp Date field:
<input type="tel" class="fs-block" maxlength="5" placeholder="Exp Date*" id="expDate" name="ExpirationDate" onkeyup="$cc.expiry.call(this,event)" onfocusout="$cc.expiryErrMsg.call(this,event)" autocomplete="off">

Whiteboard: [cc-autofill-mvp]

Updating whiteboard tag for team triage.

Whiteboard: [cc-autofill-mvp] → [ccautofill]
Priority: -- → P2
Whiteboard: [ccautofill] → [cc-autofill-mvp]
Depends on: 1647944

The severity field is not set for this bug.
:MattN, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(MattN+bmo)
Severity: -- → S3

The issue comes from how the truncation code interacts with expiration date information. The attached proof-of-concept patch (layered on top of the POC patch from Bug 1428269) demonstrates a fix based on the user's own locale (which is likely to match the website in use).

Assignee: nobody → adam
Status: NEW → ASSIGNED
Attachment #9160451 - Attachment description: Bug 1645735: Fix expiration date truncation for maxLength=(5|7) → Bug 1645735: Fix expiration date truncation for maxLength=5 r?zbraniecki!
Pushed by adam@nostrum.com:
https://hg.mozilla.org/integration/autoland/rev/203a686a3d4d
Fix expiration date truncation for maxLength=5 r=zbraniecki
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla80

Reproduced the initial issue in Beta version 79 using Windows 10.
Verified - Fixed in latest Nightly 80.0a1 (2020-07-07) (build id: 20200707094747), the expiry date is correctly formatted but the card logo is not displayed after autofill the Card Number field. Note that if I paste the same credit card number manually (copy-paste) the logo is displayed accordingly (see the video attachment below).

Status: RESOLVED → VERIFIED
Flags: needinfo?(adam)

(In reply to Alin Ilea from comment #8)

the expiry date is correctly formatted but the card logo is not displayed after autofill the Card Number field. Note that if I paste the same credit card number manually (copy-paste) the logo is displayed accordingly (see the video attachment below).

Thanks! I've filed that issue as Bug 1651123.

Flags: needinfo?(adam)
Flags: needinfo?(MattN+bmo)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: