Thickbox/Lightbox2

I'm marking as a feature request, since there's nothing that says both of these have to be enabled. But, I'd be happy to submit a patch if anyone comes up with an acceptable solution.

I have this issue too as I am using both Lightbox and Thickbox modules on my Drupal site... It would be great if the emfield video module didn't bring up both. Is it not possible in the display options for field for the content type to have an option like: Thickbox: Image Thumbnail -> Fullsize Video and Lightbox2: Image Thumbnail -> Fullsize Video, rather than the current Thickbox/Lightbox2: Image Thumbnail -> Fullsize Video? I am surprised that this is not more of an issue for other users as I have several reasons why both Light Box and Thickbox are installed. Thanks

I'm having this same issue - any workarounds?

Anything? (this is clearly a bug)

Quite surprised more people aren't reporting this.

I've dug into the module and have found this.

In emvideo.theme.inc:

function theme_emvideo_thickbox() {
...
$attributes = array(
    'attributes' => array(
      'title' => $title,
      'class' => (!isset($options['modal']) || $options['modal'] == 'thickbox') ? 'thickbox' : 'emvideo-thumbnail-replacement',
      'rel' => ((!isset($options['modal']) || $options['modal'] == 'thickbox') && module_exists('lightbox2')) ? 'lightframe['. $field['type_name'] .'|width:'. ($width + 16) .'; height:'. ($height + 16) .';]' : $field['type_name'],
    ),
    'query' => NULL,
    'fragment' => NULL,
    'absolute' => FALSE,
    'html' => TRUE,
  );

So if Thickbox/Lightbox formatter is being used and both modules are enabled, you're always going to get both popping up at once. :S
This is quite unfortunate.

In my case, simply commenting out the 'rel' => ... line is good enough (since i prefer thickbox for all video popups) but there obviously needs to be some more logic here.

I propose a setting here /admin/content/node-type/whatever/fields/field_video, that allows one to set priority between thickbox and lightbox2. Then in theme_emvideo_thickbox(), simply check for this setting and format class/rel attributes appropriately.

Changing status to bug report. Although the solution could be considered a feature request, the base problem here is, again, a bug. (unless i'm blind and am missing something)

fixed and committed to the dev release. there are now options for thickbox/lightbox2/shadowbox.

Excellent - thank you!

Confirmed that latest dev release fixes issue - closing.