479343 - Implement Upcoming Events block

Status: VERIFIED FIXED

(Websites Graveyard :: spreadfirefox.com, defect)

Description

[Alex Buchanan [:abuchanan]]

On the news/events page, the block on the lower left, the Upcoming Events still needs to be implemented.

Comment 1

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

My mock, MOZ-014_SpreadFirefox_CR3rev_Page_5.jpg, has this on the mid-right section, just below the "Quick Wins" block; is that not correct?

Comment 3

[Alex Buchanan [:abuchanan]]

correct, I have it in the right sidebar, under quick wins -> download counter -> upcoming events

MOZ-014_SpreadFirefox_CR3rev_Page_5.jpg

tomato.. wait that doesn't work when you type it...

Comment 4

[Neil Lee [:neilio]]

Attachment: Adds upcoming events HTML and CSS

Here's the changes to implement the upcoming events block. There are two new images needed which I will also attach to this bug.

Comment 5

[Neil Lee [:neilio]]

Attachment: Zipped copy of the new bg images

New bg images for upcoming events box (goes in img directory).

Comment 6

[Alex Buchanan [:abuchanan]]

i've checked in the attached patch + images to SVN in r22745

a couple things to note...

1)  the listed events aren't actual yet, Paul B. or I will have to set that up

2)  the background footer image ends in the middle of the block.  Neil is this something we can't repeat-y indefinitely?

Thanks

Comment 7

[Mary]

Loving the progress here, thanks Neil & Alex!

Is the background footer image related to the overall look and feel of the block?

It should look like a sheet of paper with shadowing behind it.  Also the gray line in between the listing and the "Upcoming Events" fads out in the design.

Attaching image of it...

Comment 8

[Mary]

Attachment: Events block design

Comment 9

[Neil Lee [:neilio]]

Attachment: Screenshot from Photoshop

Not sure what the issue is - I cut the image directly out of the mockup, so what you see is what you get, for the most part. The lines on the left and right sides of the box are very, very light, so they could just be getting lost in the difference between our displays.

Comment 10

[Mary]

Hey there:  Wanted to close the loop -- it looks fine now and it's with the same display.  So not sure what you all did :)

Alex:  Have you been able to work on getting the events to populate it?

Comment 11

[jamey]

Attachment: Upcoming Events CSS background

screenshot using xp screen resolution 1024x768 
marked with arrows missing css background images and tabs on the menu

Comment 12

[Alex Buchanan [:abuchanan]]

r22899 populates with actual events

NOTE: there is still a CSS issue with the bg image

Comment 13

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

Not sure if we want to keep bug 479343 separate from this.

Comment 14

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

(In reply to comment #13)
> Not sure if we want to keep bug 479343 separate from this.

Cut-'n-paste fail; so spammy today: really meant bug 481698.

Comment 15

[Alex Buchanan [:abuchanan]]

Neil, is there something you can do to fix the background image/css issue?  It'd be awesome if these blocks can be vertically expandable indefinitely.

If not, the alternative for this block would be to cut down the number of events entries listed.

Also, can you add some min-height to these blocks, so that when they have short/brief content they don't look offset?

Thanks!

Comment 16

[Neil Lee [:neilio]]

Attachment: Patch to fix background image issues

It's div-errific, but it works. This patch fixes the background image issue behind the events box.

Comment 17

[Neil Lee [:neilio]]

Attachment: New background images to go with above patch

Comment 18

[Alex Buchanan [:abuchanan]]

Comment on attachment 368820 [details] [diff] [review]
Patch to fix background image issues

yay

Comment 19

[Alex Buchanan [:abuchanan]]

Thank you

r23696

Comment 20

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

Verified FIXED using:

* Safari 3.2.2
* Firefox 2.0.0.20
* Firefox 3.0.7
* Firefox 3.1 beta 3
* Opera 9.64

IE 6 and 7 have issues: see bug 484925 for that.