450500 - Replicate "Hit Parade" in Affiliates Homepage

Status: VERIFIED FIXED

(Websites Graveyard :: spreadfirefox.com, defect, P1)

Description

[Juliana]

Attachment: How the Hit Parade Should Look

The "Hit Parade" on the Spread Firefox hompeage should be duplicated and posted in the Affiliates Homepage to the right of the text near the top of the page. 

Bug 450183 should be resolved before this is done - it details how the labels in the Hit Parade should be changed. 

I've attached a jpeg of how the Hit Parade should look after it has been re-labeled and duplicated.

Comment 1

[Paul Booker]

Taking this.

Comment 2

[Paul Booker]

(In reply to comment #0)
> Created an attachment (id=333645) [details]
> How the Hit Parade Should Look 
> 
> The "Hit Parade" on the Spread Firefox hompeage should be duplicated and posted
> in the Affiliates Homepage to the right of the text near the top of the page. 
> 

This can be done easily @ admin/build/block/configure/sfx_affiliates/0 by setting the "Pages" text box as follows ..

<front>
affiliates/homepage

> Bug 450183 should be resolved before this is done - it details how the labels
> in the Hit Parade should be changed.

.. i have submitted a patch for #450183 with the textual changes
 
> 
> I've attached a jpeg of how the Hit Parade should look after it has been
> re-labeled and duplicated.
> 

Comment 3

[Paul Booker]

Attachment: Included the additional text as per image.

Comment 4

[Alex Buchanan [:abuchanan]]

Comment on attachment 335505 [details] [diff] [review]
Included the additional text as per image.

the patch includes the fix for bug 450183.  please keep your patches separate for each bug.

Comment 5

[jamey]

Attachment: Hit Parade Graphics Change

Here is the Hit Parade Text change ? on the graphics if needed

Comment 6

[Alix Franquet [:alix]]

Re: comment #2
I've made the suggested change on stage and the hit parade now shows up on the affiliate page. Thanks!

Is the image change part of this bug or 450183?

Comment 7

[Paul Booker]

Yes it is . I made the initial mistake of uploading attachment 335555 [details].

Comment 8

[Alix Franquet [:alix]]

Fixed.

Comment 9

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

Verified FIXED on staging; Paul, will the changes in bug 450183 be landing soon?  I'm asking because comment 1 indicates that it should land first.

Thanks!

Comment 10

[Paul Booker]

After Alex has approved the changes ill commit the changes to subversion for further testing on stage .

Comment 11

[Alex Buchanan [:abuchanan]]

I hate to be a stickler, but let's try and keep our bugs straight in the future.

This bug had to do with adding the block to the affiliates page, let's leave it closed/fixed/verfied

bug 450183 has to do with changing the appearance of the "Top Foxes" block, so the patch and graphics belong there.  

Also, we're still missing the text, "See how Top Foxes are ranking this week."  but I'll put that in bug 450183

It's confusing when we mix these, so make sure you attach things to relevant bugs.

Thanks!