Closed (fixed)
Project:
Lightbox2
Version:
5.x-2.8
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Jul 2008 at 13:32 UTC
Updated:
8 Jan 2009 at 13:52 UTC
in the theme_lightbox2_image_ncck function (starting line 1868), $height is set
$height = $field['widget']['thumbnail_height'] === '' ? variable_get('image_ncck_default_full_height', IMAGE_NCCK_DEFAULT_FULL_HEIGHT) : $field['widget']['thumbnail_height'];
but as the title says, IMAGE_NCCK_DEFAULT_FULL_HEIGHT defaults to 0
so, at the moment, the images cant display as their height is 0. Lines 1896 and 1897 need to be updated to be
if($width) $attributes['width'] = $width;
if($height)$attributes['height'] = $height;
so they are not set if = 0; I personal would remove them all together, as defaults will still warp images of incorrect sizes.
a_c_m
Comments
Comment #1
stella commentedDone and checked in.
Cheers,
Stella
Comment #2
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.
Comment #3
stella commentedReleased in Lightbox2 6.x-1.9 and 5.x-2.9.
Cheers,
Stella