When using the fancybox with imagecache display option the theme function responsible sets the class for fancybox. This causes theme_imagecache to omit its own, usual imagecache-$presetname class, which is often used by themers.

CommentFileSizeAuthor
#1 957808-fancybox.patch1.42 KBsnufkin

Comments

snufkin’s picture

snufkin commented
Status: Active » Needs review
StatusFileSize
new 957808-fancybox.patch1.42 KB

The initial description was not entirely accurate. The issue is that when using an imagecache pre-rendered thumbnail that will open in a fancybox and use another imagecache rendered image, the thumbnail is not built using theme('imagecache', but HTML is directly built in theme_imagefield_image_imagecache_fancybox(). Which is not good, because it doesn't exactly mimic theme_imagecache, that is why there are no imagecache and imagecache-$preset classes on the resulting thumbnail.

Patch resolves it by using theme('imagecache' ... instead of directly rendering the img html in theme_imagefield_image_imagecache_fancybox.

bas.hr’s picture

bas.hr commented

Thanks! Patch is applied and change is committed.

bas.hr’s picture

bas.hr commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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