For urgent needs, I hardcoded inline.module below so that image links bear the Lightbox required rel="lightbox" option. Is it possible to make it generic in the next release of Inline?

Many thks,
P.

    if (variable_get('inline_link_img', '1')) {
      $attributes = array(
        'class' => 'inline-image-link',
        'title' => t("View") .': '. $title,
-       'rel'   => 'lightbox[gp_inline]'
      );
      $html = l($image, $file->filepath, $attributes, NULL, NULL, FALSE, TRUE);
    }
    else {
      $html = $image;
    }
CommentFileSizeAuthor
#2 jlightbox-DRUPAL-5.patch1.29 KBsun

Comments

sun’s picture

I like the idea of implementing LightBox support. But we should enable that support only if the LightBox V2 module is enabled and probably best use module_invoke() to embed the appropriate HTML which should be provided by LightBox module.

I'll be happy to commit that patch. Please see http://drupal.org/diffandpatch for more information on how to create patches.

sun’s picture

Title: Integration with Lightbox » Inline support
Project: Inline » jQuery Lightbox
Version: 5.x-1.1 » 5.x-1.x-dev
Assigned: Unassigned » sun
Status: Active » Needs review
StatusFileSize
new1.29 KB

This just needs a javascript equivalent to the one for inlined img_assist images (see js/jlightbox_image.js).

Please test attached patch.

sun’s picture

Status: Needs review » Fixed

Committed. Of course, this needs Imagecache to work.

Anonymous’s picture

Status: Fixed » Closed (fixed)