673117 - Customization page update to incorporate new add-on animation

Status: VERIFIED FIXED

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

Description

[Dan Horner]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:6.0) Gecko/20100101 Firefox/6.0
Build ID: 20110713171652

Steps to reproduce:

We have commissioned an approx 1 minute animation explaining Add-ons and their benefits Bug 646439. I would like to include this video on the customize page: http://www.mozilla.com/en-US/firefox/customize/

This video describes both Personas and Extensions, and should be added to the page so it is shown to relate to both content areas; Personas and Add-ons

The final version of animation should be ready between August 15th and 22nd

Comment 1

[Tara (musingt)]

Hmm.  John, what do you think?  we've got Personas and Add-ons split into separate sections here.  

It'd be nice to have a video box vs. a "learn more link" so it's easy to see.  But I"m not sure where it would fit within this particular layout.  Any suggestions?  Should we consider changing the layout a bit to accommodate this video?

Comment 2

[John Slater]

I'd like to avoid changing the layout as much as possible...that feels like adding extra work for not a huge amount of extra value.

Comment 3

[Tara (musingt)]

LOFO:   can you help us understand how well the add-ons section is performing in terms of generating clicks on those recommended add-ons? If it's not doing that well, I'd like to replace that graphic space with an embedded video.

Comment 4

[Dan Horner]

Hi Guys

Final version of the animation is here. Please can we implement on the customization page, but on EN pages only for the time being (universal subtitled versions to come soon)?

clients.pokelondon.com/Firefox/Firefox_final.zip
Firefox
F1R3.f0x11

Comment 5

[Tara (musingt)]

Hey guys, are we okay to simply insert the Video (with a play button on top) embedded next to the add-ons section (replacing the existing add=ons promo?)

Dan, please confirm. 
LoFo / Slater, please advise. 

Also, I wouldn't consider this a design bug if it's simply adding the video box in there. I'd change this to an implementation / website bug.

Comment 6

[John Slater]

That seems fine to me...Dan, is that what you had in mind? And yeah, if so this should be a site bug.

Comment 7

[Dan Horner]

Yep, exactly what I had in mind.

WebDev implementation bug now created:

https://bugzilla.mozilla.org/show_bug.cgi?id=689528

N.b. Final video has been updated since three weeks ago - new file linked to in implementation bug.

Comment 9

[Dan Horner]

Hey Laura / John

Looks like from a design standpoint we're agreed and ready to go (see bug 673119)

Please can we move ahead and implement? 

Two additional requests (let me know if I need to create new bugs)

1. Please can we make sure plays on this video is tracked (per locale)
2. Can we make sure that the animation, when implemented, is compatible with universal subs? (Please see bug 682336)

Thanks

Comment 10

[Laura Forrest]

Steven - Can you take a look at this for the 4.1 release? Let Dan or I know if you have any questions; this 

Brad - Looks like DCS tracking is needed here on the "play" button. Can you provide a link to, or the exact tagging to use?

Comment 11

[Bradley Gross]

In the anchor tag that controls the video plays please include the following dcsMultiTrack call in the onClick event handler:

dcsMultiTrack(
'DCS.dcssip','www.mozilla.org',
'DCS.dcsuri','/en-US/firefox/features/',
'WT.ti','Video: <insert video name>',
'WT.dl',99
)

Comment 12

[Laura Forrest]

Steven - This is ready to be pushed when you are. 

Please add the tracking in comment 11 as well. Thanks!

Comment 13

[Steven Garrity [:sgarrity]]

We'll need to get the video uploaded to videos.mozilla.org. Dan, can you file an IT bug for them to upload the video? Then we can get it integrated into the page.

Comment 14

[Dan Horner]

hi Steven, the video has already been uploaded (in multiple formats) by Matthew Zeier (see upload bug 696470)

Location: http://videos.mozilla.org/serv/marketing/696470/

Will that do?

Comment 15

[Laura Forrest]

Steven - Is there anything else that's needed? Moving this to 4.5 just so it's not left behind in 4.4, but please push this live when it's ready.

Comment 16

[Laura Forrest]

Hey Steven - A friendly ping on this. Anything else you need?

Comment 17

[Steven Garrity [:sgarrity]]

I've added the video in trunk.

(In reply to Bradley Gross from comment #11)
> In the anchor tag that controls the video plays please include the following
> dcsMultiTrack call in the onClick event handler: ...

Brad, we have a <video> tag on this page for browsers that can display it, and fall back to a Flash player. On the video tag, we us JS to create a transparent link (with the play button) that overlays the player to start the video. This is done with a site wide video setup script. Is there a way we could have that script setup a dcsMultiTrack() for all videos on the site?

I don't think it's possible to do on the Flash player?

Comment 18

[Steven Garrity [:sgarrity]]

I'm closing this so the video can get pushed in the 4.7 release. Still waiting in feedback on the <video> ta webtrends issue in comment #17. That might deserve a separate bug.

Comment 19

[Bradley Gross]

Ack, this fell off my radar.  The short answer is yes, you can invoke a dcsMultiTrack call on the transparent link.  If you are thinking of doing this site wide, you would just need to dynamically populate whichever parameter is storing the movie name.  I'm happy to provide assisstance in creating that tag.

Comment 20

[James Long (:jlongster)]

(In reply to Steven Garrity from comment #18)
> I'm closing this so the video can get pushed in the 4.7 release. Still
> waiting in feedback on the <video> ta webtrends issue in comment #17. That
> might deserve a separate bug.

I had a large conflict with stage. Steven, do you mind pushing it to stage? I'm not sure how to resolve it. If it's not any easier for you I can figure it out.

Comment 21

[James Long (:jlongster)]

Actually, it wasn't that hard, I figured it out. pushed in r98022.

Comment 22

[raymond [:retornam] (inactive)]

verified fixed http://www.mozilla.org/en-US/firefox/customize/index.html