The img_assist module has a popup window for selecting and inserting image nodes into other content. This select window displays a number of image thumbnails with links to the image node that is to be selected and inserted. Because of this, Lightbox2 thinks that it needs to rewrite the link, which heavily interferes with img_assist's image selection process.

One possible solution would be to disable the Lightbox2 scripts on the img_assist pages. The selection page has a body id of "img_assist_thumbs" and a body class of "img_assist", if that helps at all.

Comments

stella’s picture

stella commented
Assigned: Unassigned » stella
Status: Active » Fixed

Patches have been added to CVS to prevent it from interfering with the images in the img_assist pop-up window. You can either download the latest version from CVS or switch off image node support on the settings page. It will be included in the next release.

Cheers,
Stella

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)
ashtonium’s picture

ashtonium commented
Version: 5.x-1.1 » 5.x-1.x-dev
Status: Closed (fixed) » Needs work

First off, thanks for your quick response to the original posting. I've finally gotten time to test the 5.x-1.x-dev version, and it isn't fixing the issue.

I've looked at your changes, and they seem like they should fix the problem, but the issue still exists. I think this may be because Element.hasClassName wants an id string as the first value, at least according to the example page: http://wiki.script.aculo.us/scriptaculous/show/Element.hasClassName

I tested out
if (!Element.hasClassName('img_assist_thumbs', "img_assist")) {
which does work, but there may be a better way to do it, as I'm no javascript guru.

stella’s picture

stella commented
Status: Needs work » Fixed

Thanks for that. I've corrected this for both the drupal 5.x and 4.7.x versions and have checked them into CVS. They will be included in the next release.

Cheers,
Stella

ashtonium’s picture

ashtonium commented

Looks good, seems to be working very well now.

Just as a side note, I don't think this line is needed anymore either:
var body = document.getElementsByTagName("body");

since we're not passing the body element to Element.hasClassName anymore.

Thanks again for you're quick responses,
-Ashtonium

stella’s picture

stella commented

done, thanks.

stella’s picture

stella commented

Marked #151292 as a duplicate of this issue.

Anonymous’s picture

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