please QA the Phishing Protection page on the new Mozilla.com

VERIFIED FIXED in 3.0

Status

VERIFIED FIXED
11 years ago
6 years ago

People

(Reporter: jslater, Assigned: stephend)

Tracking

unspecified

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

11 years ago
Hi Stephen. Please QA as discussed.

Thanks,
John
(Assignee)

Comment 1

11 years ago
The only thing I see is:

1) When JavaScript is disabled, the expand/collapse widgets disappear, and the heading doesn't shift left, so there's an empty space (minor UI nit)

(Page validates and all links work, so if we're not too concerned about the issue above we're good to go and you can mark it fixed; the missing artwork I'll track in a yet-to-be-created asset-checking bug.)
What about Malware Protection?
Stephen - the alignment issues when JS is disabled is fixed.

Reed: ?
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
(Assignee)

Comment 4

11 years ago
Verified FIXED
Status: RESOLVED → VERIFIED
(Reporter)

Comment 5

11 years ago
Quick update on this...I was talking to Mike Beltzner today and there are some likely changes to the functionality of the phishing and malware changes that could affect the content here a bit. 

He'll be chiming in soon with the details and background...stay tuned.
(Reporter)

Comment 6

11 years ago
Another update on this: I just filed Bug 432458 to incorporate the copy edits referred to in comment #5 above. As soon as they're complete, we should re-QA this page to make sure everything is working properly.

I'll follow up with Pascal about l10n issues in a separate thread.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
(Assignee)

Updated

11 years ago
Depends on: 432458
(Assignee)

Comment 7

11 years ago
1) Broken link: "ttp://code.google.com/p/google-safe-browsing/wiki/Protocolv2Spec"
2) "Attackers will often hack a site to turn it into an attack site" -- "attack site" should be capitalized to be consistent.
3) Page doesn't validate
4) On Linux, the instructions read, "Tools > Options ... > Security", which should be "Edit > Preferences > Security"--we've got Linux correct on other pages like https://www-firefox3.authstage.mozilla.com/en-US/firefox/tips/
(Assignee)

Comment 8

11 years ago
5) Can we tighten up the vertical footer space again?
(In reply to comment #7)
> 1) Broken link:
> 2) "attack site" should be capitalized to be consistent.
> 3) Page doesn't validate

1, 2, and 3 fixed in r12946.

> 4) On Linux, the instructions read, "Tools > Options ... > Security", which
> should be "Edit > Preferences > Security"--we've got Linux correct on other
> pages like https://www-firefox3.authstage.mozilla.com/en-US/firefox/tips/

Added Linux-specific menu location. I left off the ellipses after Options for Linux as it doesn't seem to be there in the menus.

I also replaced the three periods with a proper ellipses character (and removed the space before it) in all cases.

As for the vertical footer space, I think this is as tight as we can get it without the gradient looking too abrupt on shorter pages (with no scrolling).
Status: REOPENED → RESOLVED
Last Resolved: 11 years ago11 years ago
Resolution: --- → FIXED
(Assignee)

Comment 10

11 years ago
6) Sorry Steven, I missed one, "You can learn more about attack sites" (capitalize).
7) "Firefox 3 or later contains a built-in Phishing and Malware Protection to help keep you safe online. These features..." -- does it seem odd to anyone else that it's singular ("a") in the first sentence, and plural "These features..." in the next?  Can we just drop the indefinite article "a"?
(In reply to comment #10)
> 6) Sorry Steven, I missed one, "You can learn more about attack sites"
> (capitalize).
> 7) "Firefox 3 or later contains a built-in Phishing and Malware Protection to
> help keep you safe online. These features..." -- does it seem odd to anyone
> else that it's singular ("a") in the first sentence, and plural "These
> features..." in the next?  Can we just drop the indefinite article "a"?

Yep, that's an oopsie on my part.  I realised while writing copy that treated it as a single feature, that it was really two features, and went through to refer to them in the plural.  Looks like I missed one.  Nice catch.
(In reply to comment #10)
> 6) Sorry Steven, I missed one, "You can learn more about attack sites"
> (capitalize).
> 7) "Firefox 3 or later contains a built-in Phishing and Malware Protection to
> help keep you safe online. These features..." -- does it seem odd to anyone
> else that it's singular ("a") in the first sentence, and plural "These
> features..." in the next?  Can we just drop the indefinite article "a"?

Both done in r13014.
(Assignee)

Comment 13

11 years ago
Argh.

8) (?): "Attackers will often hack a site to turn it into an Attack Site, and sometimes the web site's owner won't even know that this has happened." -- capitalize "web", and we're done here?  Sorry for the iteration. :-(
(Assignee)

Comment 14

11 years ago
I should've done a CTRL+F; there are other instances of s/web/Web:

9) "Phishing attacks usually come from email messages that attempt to lure the recipient into updating their personal information on fake, but very real looking, web sites."
10) "Attack Sites are web sites that try to infect your computer with malware when you visit."

Steven, again, sorry for missing these.
(In reply to comment #14)
> I should've done a CTRL+F; there are other instances of s/web/Web:
Got these in r13082.
(Assignee)

Comment 16

11 years ago
Verified FIXED; thanks again for your patience.
Status: RESOLVED → VERIFIED
Component: www.mozilla.org/firefox → www.mozilla.org
Product: Websites → Websites
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in before you can comment on or make changes to this bug.