Closed Bug 1332901 Opened 3 years ago Closed 3 years ago

Land revised strings for the insecure login field contextual warning in 53

Categories

(Toolkit :: Password Manager, defect)

53 Branch
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox52 --- wontfix
firefox53 --- fixed

People

(Reporter: MattN, Assigned: MattN)

References

Details

Attachments

(1 file)

(Quoting Tanvi Vyas [:tanvi] from bug 1319176 comment #17)
> Spoke to Ryan today and we decided to add "Learn More" to end the of string,
> with no elipse or period.  The Learn More should be styled slightly
> differently - with medium text.  Selecting the warning row should open the
> sumo learn more page.

Ryan also wants to try remove the first sentence from the warning.

We need to re-land the Learn More string that's already on Aurora and make it so that Learn More can go at the end of the existing string instead of on its own line.
Comment on attachment 8829202 [details]
Bug 1332901 - Land revised strings for the insecure login field contextual warning in 53.

rs=tanvi over IRC
Attachment #8829202 - Flags: review?(tanvi) → review+
Pushed by mozilla@noorenberghe.ca:
https://hg.mozilla.org/integration/mozilla-inbound/rev/80d47de5e1bf
Land revised strings for the insecure login field contextual warning in 53. r=tanvi
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e353d77bb730
Add back string which is still in use (new ones got added but not yet used). r=fix
https://hg.mozilla.org/mozilla-central/rev/80d47de5e1bf
https://hg.mozilla.org/mozilla-central/rev/e353d77bb730
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.