423990 - Track and replace FPO artwork with final images

Status: VERIFIED FIXED

(www.mozilla.org :: General, defect)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

Tracking bug for replacing the FPO artwork with the final images.

I think if we just inline-list each page with which artwork needs replacing, we can best track the replacement and testing of the FPO->final artwork transition.

Comment 1

[Stephen Donner [:stephend] Not actively reading bugmail]

1) From https://www-firefox3.stage.mozilla.com/en-US/firefox/tips/:

https://www-firefox3.stage.mozilla.com/en-US/img/tignish/tips/advanced-02-mac.jpg
https://www-firefox3.stage.mozilla.com/en-US/img/tignish/tips/advanced-04-mac.jpg
https://www-firefox3.stage.mozilla.com/en-US/img/tignish/tips/advanced-05-mac.jpg

2) From https://www-firefox3.stage.mozilla.com/en-US/press/:

https://www-firefox3.stage.mozilla.com/img/tignish/press/sources-reviewers-guide.png

Steven/John - please feel free to directly add as you see/have time (I'm still a "bit" behind, as you probably have noticed).

Comment 2

[Stephen Donner [:stephend] Not actively reading bugmail]

3) More from https://www-firefox3.stage.mozilla.com/en-US/firefox/tips/:

https://www-firefox3.stage.mozilla.com/en-US/img/tignish/tips/advanced-02.jpg
https://www-firefox3.stage.mozilla.com/en-US/img/tignish/tips/advanced-04.jpg
https://www-firefox3.stage.mozilla.com/en-US/img/tignish/tips/advanced-05.jpg

(These really are known, but I didn't want this bug to go to waste! :-P)

Comment 3

[Steven Garrity [:sgarrity]]

(In reply to comment #1)
https://www-firefox3.stage.mozilla.com/img/tignish/press/sources-reviewers-guide.png
This one is fixed in r12222.

Comment 4

[Steven Garrity [:sgarrity]]

Everything from the Tips page except:
/en-US/img/tignish/tips/forgery.png
/en-US/img/tignish/tips/advances-04.jpg
/en-US/img/tignish/tips/advances-04-mac.jpg

are fixed in r13249.

Comment 5

[Steven Garrity [:sgarrity]]

Features page images are done in r13254.

Comment 6

[Steven Garrity [:sgarrity]]

These two are done in r13258:
/en-US/img/tignish/tips/advances-04.jpg
/en-US/img/tignish/tips/advances-04-mac.jpg

Tips page just needs /en-US/img/tignish/tips/forgery.png updated now.

Comment 7

[Steven Garrity [:sgarrity]]

Security and forgery screenshots on the tips page are done in r13342 and r13343. Tips page should be all-clear.

Comment 8

[Stephen Donner [:stephend] Not actively reading bugmail]

Vista isn't showing me the forgery images; is it due to this?:

else if (gPlatformVista) {
  document.write('<img src="/en-US/img/tignish/tips/forgery-vista.png" alt="Forgery screenshot" />');

If I do javascript:alert(gPlatformVista); in the location bar, that global var is undefined; (gPlatform==1, though.)

This is post-fix of 433804.

Comment 9

[Steven Garrity [:sgarrity]]

(In reply to comment #8)
> Vista isn't showing me the forgery images; is it due to this?:...

Forgot to commit one of the JS files. The vista check should work in a few minutes. Thanks.

Comment 10

[Stephen Donner [:stephend] Not actively reading bugmail]

Thanks, Steven; I've checked and verified that comment 9 is fixed.  John, Steven, others: is there anything left to do?  I've done a run through on en-US and nothing's jumping out at me...

Comment 11

[John Slater]

(In reply to comment #10)
> Thanks, Steven; I've checked and verified that comment 9 is fixed.  John,
> Steven, others: is there anything left to do?  I've done a run through on en-US
> and nothing's jumping out at me...
Not that I know of...I know the Royal Order (and Steven) made a bunch of other updates to screenshots not referenced in this bug so we should be pretty well-set. Small inconsistencies might come up after launch, but there's nothing major left here. 

Marking this one as fixed.

Comment 12

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED; I double-checked this again tonight, yay!