Yay! for this superb module, and its first issue.. :)

The hover-preview removes the 'linked-to-node' imagecache formatter. is there a way by which this can be achieved too?

Btw, I am modifying hover-preview to use it for swappable rollover images (minor jquery alteration). let me know how would you like to integrate this into the module already built?

CommentFileSizeAuthor
#4 hover_preview.js_.txt683 bytesmesh
#1 hover_preview-linked-node.patch1.15 KBmesh

Comments

mesh’s picture

StatusFileSize
new1.15 KB

partly answered. but i dont know how do convert this as an optional formatter.

mrfelton’s picture

I'm looking to do swappable rollover images too (make an image greyscale when hovered over). Is this possible with the patch at #1?

rjbrown99’s picture

Thanks for the posts and input. So you are liking the module and formatter approach but want to use it for different Javascript effects? Perhaps something of a pluggable effect system? That might work for one effect at a time, but if you had two at once the number of presets would start to get out of hand.

I haven't looked at the patch yet but I'll review this weekend.

mesh’s picture

StatusFileSize
new683 bytes

@mrfelton - the patch provided is for keeping the imagecache thumbnail linked to the node - it has nothing to do with the swapping of images. actually the hover is done by jquery, which is changed to swap image 'src'. see the attached .js, and add to it :)

mesh’s picture

@rjbrown99 - yes, sort of. it would be great if the user could select what effect, 'swap', or 'hover' (maybe there can be more?)
I understand that the number of presets (formatters) will exceed, but then they can be grouped into the list (ugly way), or can be provided as modules separately?

rjbrown99’s picture

Title: Linked to node? » Implement ability to select different javascript effects
Category: support » feature

I'm going to mark this as a feature request. I'm not sure if that's the direction I want to go personally but I'd like to hear if others are interested in this, and if so thoughts as to how to implement effectively.

David Hernández’s picture

Subscribing.
Will be great to handle multiple javascripts effects as hover and lightbox2. The problem will be the huge list of options with only a few different presets. I have some code done but just to add the options of the lightbox and hover but not handle the javascript and at the moment I don't have enough time to do it :(

mrfelton’s picture

Wouldn't it be better if these presets were set up on request, rather than just setting one up for every since set of possible options..?

David Hernández’s picture

@mrfelton, I think your approach is the best solution if we don't want to show a huge list of options.

mesh’s picture

How about a module settings page, asking the user to enable features (as simple checkboxes) ? this page could also include simple demos of what each feature provides.

rjbrown99’s picture

Status: Active » Postponed

Patches welcome. I don't need this feature myself, but I'm happy to review and/or evaluate enhancements. Until then I'm flagging postponed.

sballinger26’s picture

hey mesh
Thanks a lot ur code works gr8 for me.........
u r god for me im looking for this code from last 3 days....
at last i found here...
thanks once again

enjoy

mesh’s picture

@sballinger26: good to know that the code helped..

rjbrown99’s picture

#13: Can you explain a bit more as to what your code is doing? Do you view this as a specific fix for your situation or a broader fix that would apply to all users of the module?

It seems like we have a positive review in #12. With a bit more information I can commit this to the -dev branch at least so we can get more people to give it a whirl.

rjbrown99’s picture

Status: Postponed » Needs review
mesh’s picture

@rjbrown99: the code modifies the default behavior of hover preview and converts it to swappable rollover images. I will setup a demo site soon to show what it does. As mentioned earlier I need your suggestions on how to integrate this into the hover-preview module, or if it can become a separate module? (with a dependency on hover-preview of course).

mesh’s picture

Status: Postponed (maintainer needs more info) » Needs review

i have set up a demo site here - http://sandbox.openmap.in The demo site is not active any more, sorry.

rjbrown99’s picture

#17:

For selectable effects, I'd suggest one of two paths.

1. The Libraries module provides a way to include extra libraries. This could be the home for any effects, and this module could detect supported effects and allow switching them via an admin interface.

2. The jQuery Plugin module provides a different way to do it. Same idea, this could host the supported effects.

In all cases, this isn't functionality I need for my site so I'm throwing this out there as an option if you wish to develop it. I'm happy to review it or consider it for inclusion.

uno’s picture

#4 - This is exactly what I was looking for and the reason why I installed Hover Preview module in the first place. I really hope that this swap image option could somehow be a part of the module itself.

Anyhow, thanks to both of you :)

rjbrown99’s picture

Status: Needs review » Postponed (maintainer needs more info)
mesh’s picture

Status: Needs review » Postponed (maintainer needs more info)

@rjbrown99: let me know how we can get this thing done further, and what info you need.

rjbrown99’s picture

#21 - Per comment #11 above I'm happy to continue to chat about implementation approaches and provide feedback but I have no need for this myself - i.e. I am not planning to write any code to support it. What would be needed would be someone to write the code and submit it to the issue as a patch for review, either as a .patch or via a sandbox project.

robloach’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Postponed (maintainer needs more info) » Active

Might actually be useful... Definitely doable in Drupal 7 with the Display Settings.

robloach’s picture

Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Active » Patch (to be ported)
naeluh’s picture

@mesh

Hi i altered the hover_preview.js to reflect number #4

Posted by mesh on February 18, 2010 at 10:19am
@mrfelton - the patch provided is for keeping the imagecache thumbnail linked to the node - it has nothing to do with the swapping of images. actually the hover is done by jquery, which is changed to swap image 'src'. see the attached .js, and add to it :)

Attachment Size
hover_preview.js_.txt 683 bytes

when I rollover the image instead of the imagecache image I get a blank box that says the - img src="[object HTMLImageElement]"

Please gimme a hand I really am interested in using this thanks !

You can see it here http://50.56.121.20/web

naeluh’s picture

@rjbrown99

Hi do you have any insite as to why this would be happening ?

thanks
Nick

naeluh’s picture

Priority: Normal » Major
rjbrown99’s picture

Priority: Major » Normal

Please reserve the use of priority for issues that impact the module's operation for a number of users. This is a feature request for enhancements to Javascript, which would not be a major issue.

naeluh’s picture

My apologies is there some documentation I can look at so I can further my efforts to fix my issue
I am just want to learn how and I am hitting a wall thanks for the help I really appreciate it

open social’s picture

4# Works nice!

sarhansg’s picture

Will this feature be added to this module? If not could someone piece together instructions on how this can be achieved. Would be very helpful for newbs to Drupal.

open social’s picture

Have you tried the patch in #4?

sarhansg’s picture

No, I have not as my site is in a live/production environment. Might set up a test site to play around with it.

Andre-B’s picture

Issue summary: View changes
Status: Patch (to be ported) » Closed (outdated)

Drupal 6 support ended on February 24th 2016. https://www.drupal.org/drupal-6-eol