Hi there,

First, love the module - it works nicely. I went with the 1.1 version, which did not allow ESC to close the image, then upgraded to your June 6 dev version, in which ESC does work.

I looked in the js folder of the dev version and noticed you are still using Lightbox.js version 2.02, which is several revs back. As of this writing, the current version of Lightbox V2 is 2.03.3 - any chance you can include that as part of the next dev release? Are there any issues you've had which would prevent you from using the latest code? I tried just replacing the lightbox.js file with 2.03.3 and the module broke.

The one beef I have with Lightbox is that the window opens and pushes the close box just below the bottom of the window. When I reduced the size of my "preview" images in image.module to 600 pixels, the window still shows full size. This was more of an issue when ESC didn't work - the user would have to scroll down to click the close button every time. Is there a way to specify the size of the window that appears (perhaps as a percentage of the screen size) so it won't cover up the close button?

Latest Lightbox: http://www.huddletogether.com/projects/lightbox2/releases/lightbox2.03.3...

Thanks VERY much - again, great work!

John @ TurboChef

Comments

TurboChef’s picture

No comment so far? Anyone home?

stella’s picture

Sorry, I've been rather busy.

I think I might have updated the lightbox.js in the lightbox2 module with the latest stuff from huddletogether.com when I did the last release but forgot to update the version number. I haven't had the chance yet to go through the code to verify this. Do you know of any features that aren't in the lightbox2 module?

Stella

stella’s picture

FYI: Marked #159604 as a duplicate of this issue.

dilcher’s picture

Is there a fix for this? I tried following the "duplicate of xxxxx" link, but couldn't find a solution....

stella’s picture

Assigned: Unassigned » stella

For the issue where when you open a large image and the lightbox appears but pushes the close button below the bottom of the browser window, make the following change to line 72 of lightbox2/js/lightbox.js

var borderSize = 18;

I've added this to CVS and it will be included in the next release.

Cheers,
Stella

stella’s picture

Status: Active » Fixed

This has been released in 5.x-2.0 and 6.x-1.0. It now uses jQuery instead of prototype/scriptaculous and is a port of the latest version (2.03.3) from http://www.huddletogether.com

Cheers,
Stella

Anonymous’s picture

Status: Fixed » Closed (fixed)