As asked by Kieran, I've developed a jquery plugin that presents a slideshow from a flickr photoset. The aim of this work is to add a slideshow with photos of people on drupalcons, on the mission statement at boston2008.drupalcon.org site.

It reads from this photoset:

http://www.flickr.com/photos/23181861@N08/sets/72157603787442877/

Patch rolled against bluebeach theme of boston2008.drupalcon.org site attached.

CommentFileSizeAuthor
#11 mission_block.diff1.72 KBnnewton
#7 800x600.png342.56 KBmikey_p
#6 600x800-safari.png177.07 KBAmazon
#6 1280*854-safari.png201.27 KBAmazon
#5 drupalcon.org_.png354.07 KBmikey_p
#4 Screenshot-Drupalcon Boston 2008 - Mozilla Firefox.png443.3 KBchrisfromredfin
bluebeach_slideshow.patch5.96 KBrecidive

Comments

Amazon’s picture

Amazon commented

Modifying BlueBeach has become a problem for us. If this is to be done as a module, which modifies the mission statement then please provide the module.

Ideally we would try this patch on Drupalcon.org and ultimately have this same feature in the mission statement on Drupal.org to help with showcases.

Kieran

recidive’s picture

recidive commented

A module can provide a block, changing the mission statement would require theme modifications AFAIK.

Amazon’s picture

Amazon commented
Status: Needs review » Needs work

This has now been deployed at http://drupalcon.org. Thanks Henrique!

Henrique, if you get a chance to make the speed variable and add a pause on mouse over, or click to start slide show that would help to get this patch on Drupal.org.

Kieran

chrisfromredfin’s picture

chrisfromredfin
he/him/his
Primary language English
Location Portland, Maine
commented
StatusFileSize
new Screenshot-Drupalcon Boston 2008 - Mozilla Firefox.png443.3 KB

Linux Firefox seeing the slideshow appear this way (see attachment):

mikey_p’s picture

mikey_p commented
StatusFileSize
new drupalcon.org_.png354.07 KB

Rather broken in Safari 3 as well. See attached.

Amazon’s picture

Amazon commented
StatusFileSize
new 1280*854-safari.png201.27 KB
new 600x800-safari.png177.07 KB

600*800 in safari 3.0.4 looks fine.
1280*854 in safari 3.0.4 looks fine.
Both on leopard osx.

mikey_p’s picture

mikey_p commented
StatusFileSize
new 800x600.png342.56 KB

Amazon - January 26, 2008 - 13:58
600*800 in safari 3.0.4 looks fine.
1280*854 in safari 3.0.4 looks fine.
Both on leopard osx.

On 10.5 here, with Safari Version 3.0.4 (5523.10.6)

Above screenshot was with 1280x800.
I've attached another with 800x600, same problem

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

be sure to refresh your browser cache. I had this problem on Firefox+Ubuntu as well at first - a "ctrl+f5" fixed it for me.

kkaefer’s picture

kkaefer commented

The file name of the CSS file should be changed (or postfixed with a unique string) so that all browsers load the new file. I believe drupal.org uses far future expires headers for all static content so the fixed slideshow won't show up in browsers which already have the old, and broken version cached.

chrisfromredfin’s picture

chrisfromredfin
he/him/his
Primary language English
Location Portland, Maine
commented

The mission statement text seems unnecessarily large (by comparison with standard text). It's already highlighted by use of the orange box, so I wonder if shrinking the missiontext font size by a point or so would push the other content up (like "sponsorships," which is obviously important).

Also, I'm seeing that the "a" tags around the first link seem a little peculiar to me. Right now they're like:

<a>Register, then pay $195 conference</a><a>fee</a>

I think it would be much more sensible to use:

<a>Register</a>, then <a>pay $195 conference fee</a>
nnewton’s picture

nnewton commented
StatusFileSize
new mission_block.diff1.72 KB

Attached is the diff for the modified mission block template (what is displaying the table of links).

chrisfromredfin’s picture

chrisfromredfin
he/him/his
Primary language English
Location Portland, Maine
commented

This patch doesn't address comment #10:

+  <td id="ml1"><a href="user/register">Register, then pay $195 
conference <a href="fee">fee</a></td>

...note there's no closing a-tag before the href="fee" starts.

recidive’s picture

recidive commented

IMO, these links could go direct on the mission statement configurable at admin/settings/site-information. There is no reason to hardcode this in the theme.

The 'style' tags are not acceptable and we should consider moving this to the .css file. Also we should consider doing this as unordered lists in a similar way to how its done on drupal.org.

By the way, can we move this discussion to another issue, and keep this one focused on the focus slideshow?

nnewton’s picture

nnewton commented

re: cwells

Yes, the patch just reflects what was currently on the site, not my proposal for fixing anything. I've since fixed the closing anchor. Thanks for pointing that out.

re: recidive

I agree with pretty much everything you say, the current version is just a hack job Kieran and I threw together when nobody else was online. New issue to fix it here: http://drupal.org/node/215220

recidive’s picture

recidive commented

We need to change slideshow/slideshow.js to point to this new photoset:

http://www.flickr.com/photos/24208587@N04/sets/72157604052983535

Anyone with access to server could change the 'photoset_id' param to 72157604052983535 for us?

Thanks!

mikey_p’s picture

mikey_p commented
Status: Needs work » Fixed
Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.