[Form Autofill] Make Phishing Warning consistent no matter hovering on items or not.

RESOLVED FIXED in Firefox 56

Status

()

defect
P3
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: selee, Assigned: ralin)

Tracking

(Blocks 1 bug)

unspecified
mozilla56
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox56 fixed)

Details

(Whiteboard: [form autofill:M4], )

Attachments

(2 attachments)

Phishing warning is changed after hovering on a profile item.
When focusing on address-level2 field, the phishing warning shows "Also fill company, address, phone, email".

Then hovering on any item, it shows "Also fill company, phone, email" (without "address").

There should be no difference of wanring message when hovering items or not.
Yeah, I agree we should be consistent about whether "address" is shown in that case.
(In reply to Matthew N. [:MattN] (huge backlog; PM if requests are blocking you) from comment #1)
> Yeah, I agree we should be consistent about whether "address" is shown in
> that case.

I'd prefer not to show "address" since it has already focused on ddress-level2 in this case. So, both case(selected or non-selected) would show the same string: "Also fill company, phone, email"

MattN, do you think it's fine to do that to minimize the chance of height changing?
Flags: needinfo?(MattN+bmo)
Hey Ray,

(In reply to Ray Lin[:ralin] from comment #2)
> I'd prefer not to show "address" since it has already focused on
> ddress-level2 in this case. So, both case(selected or non-selected) would
> show the same string: "Also fill company, phone, email"

After discussing with Juwei, she thinks that the warning message should not contain the focused type and be consistent.
Another issue is about the order of the field type in Warning Message.
Please follow the item 3 in the spec[1] to design the message. Thanks.

[1] https://mozilla.invisionapp.com/share/AP8TFZ22G#/screens/185446489
Assignee: nobody → ralin
Status: NEW → ASSIGNED
(In reply to Sean Lee [:seanlee][:weilonge] from comment #3)
> Hey Ray,
> 
> (In reply to Ray Lin[:ralin] from comment #2)
> > I'd prefer not to show "address" since it has already focused on
> > ddress-level2 in this case. So, both case(selected or non-selected) would
> > show the same string: "Also fill company, phone, email"
> 
> After discussing with Juwei, she thinks that the warning message should not
> contain the focused type and be consistent.
> Another issue is about the order of the field type in Warning Message.
> Please follow the item 3 in the spec[1] to design the message. Thanks.
> 
> [1] https://mozilla.invisionapp.com/share/AP8TFZ22G#/screens/185446489

I've addressed that problem in the patch, thanks for the information.
Flags: needinfo?(MattN+bmo)
Comment on attachment 8885641 [details]
Bug 1379575 - Do not show focused category in autofill notes if none of the profiles have been selected.

https://reviewboard.mozilla.org/r/156496/#review161678
Attachment #8885641 - Flags: review?(MattN+bmo) → review+
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8f07c253160b
Do not show focused category in autofill notes if none of the profiles have been selected. r=MattN
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/8f07c253160b
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.