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

Status: RESOLVED FIXED

(Toolkit :: Password Manager, defect)

Description

[Matthew N. [:MattN]]

(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 1

[Matthew N. [:MattN]]

Attachment: Bug 1332901 - Land revised strings for the insecure login field contextual warning in 53.

Review commit: https://reviewboard.mozilla.org/r/106356/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/106356/

Comment 2

[Matthew N. [:MattN]]

Comment on attachment 8829202 [details]
Bug 1332901 - Land revised strings for the insecure login field contextual warning in 53.

rs=tanvi over IRC

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/80d47de5e1bf
Land revised strings for the insecure login field contextual warning in 53. r=tanvi

Comment 4

[Pulsebot]

Pushed by [e-mail]:
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

Comment 5

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/80d47de5e1bf
https://hg.mozilla.org/mozilla-central/rev/e353d77bb730