The code sets a height CSS property for each gallery box (ie, in the list on on /image).
This means that more than about a sentence of description text for a gallery makes the text runs off the box

This would at least be better given as a relative measurement in ems (or ex, since it's vertical).
I'd actually suggest removing the height completely.

(image_gallery.module line 257)

CommentFileSizeAuthor
#5 136338.patch551 bytesHetta
#2 image_gallery_gallerylistheight.patch653 bytesjoachim

Comments

drewish’s picture

drewish commented

roll a patch.

joachim’s picture

joachim commented
StatusFileSize
new image_gallery_gallerylistheight.patch653 bytes

There's an extreme case where the image spills out of the box -- either the text is tiny or you've got pretty big thumbnails.
To cover this, I've applied drupal's built-in clear-block class.

drewish’s picture

drewish commented
Status: Active » Needs work

it looks like that patch was against an older version of the file. take a look at the current code and let me know if this is still applicable.

drewish’s picture

drewish commented

marked http://drupal.org/node/164352 as a duplicate

Hetta’s picture

Hetta commented
Version: 5.x-1.0 » 5.x-1.x-dev
StatusFileSize
new 136338.patch551 bytes

Joakim's patch in #2 changes a style which isn't found in the current module
$content .= '<li style="height : '.$height .'px">';
(currently $content .= '<li>';)
to
$content .= '<li class="clear-block">'; )
on line 336.
In case that's necessary for themes: here's a patch against the current dev (Aug 02, 2007).

drewish’s picture

drewish commented
Status: Needs work » Fixed

Committed to HEAD and DRUPAL-5.

Anonymous’s picture

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