Bug 16907 – Lightbox-based Image Gallery Breaks

• Component: New Bugs
• Status: RESOLVED INVALID

[Sean McMains]

We have an image gallery based on the Lightbox Javascript that worked beautifully under Safari 2.x, and works correctly with IE and Firefox. Safari 3.x, however, appears to use the width of the first image displayed for all subsequent images, regardless of their actual size.

To reproduce:

1. Go to http://www.theatreanddance.txstate.edu/productions/past/0607/markham.html
2. Click one of the production photos with a wide aspect ratio
3. Use the Next and Previous links to see a photo with a tall aspect ratio

Expected behavior:
The image should be displayed at its actual size

Actual behavior:
The image is scaled to match the width of the first image.

[Sean McMains]

Created attachment 18502 [details]
Gallery Image in Safari

Here's what it looks like in Safari

[Sean McMains]

Created attachment 18503 [details]
Gallery Image in Firefox

Here's the correct rendering. It works fine in Firefox, Safari 2, and IE.

[David Kilzer (:ddkilzer)]

I'm seeing the bad behavior on Safari 2.0.4 (419.3) with original WebKit on Mac OS X 10.4.11 (8S165).

Which version of Safari 2 and OS X are you using?

The latest Lightbox Image Viewer (v2.03a) seems to work for me with a nightly WebKit build and Safari 3 with their local demo.  Have you tried upgrading to see if that fixes the issue?

http://www.dynamicdrive.com/dynamicindex4/lightbox2/index.htm

[Sean McMains]

That's bizarre. I know that things used to be working fine with Safari 2.0.x. Perhaps one of the updates to Prototype or Scriptaculous triggered the problem with Safari. (The versions Lightbox comes bundled with are woefully out of date now.)

I'll do some experimentation and will update with the results. Thanks!

[Sean McMains]

Mea culpa.

It looks like this was a problem with our code which was changed around the same time that Safari 3.0 was released, as older versions of our stuff work fine with Safari 3.0. We're working on tracking it down on our end now.

Sorry for the false alarm!

[Sean McMains]

For future generations: 

This turns out to be an issue with the more recent versions of Prototype and Scriptaculous. A fix to the lightbox.js is posted here:

http://www.huddletogether.com/forum/comments.php?DiscussionID=1836&page=1#Comment_6626

[David Kilzer (:ddkilzer)]

Thanks for doing the research and reporting back, Sean!