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?
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | hover_preview.js_.txt | 683 bytes | mesh |
| #1 | hover_preview-linked-node.patch | 1.15 KB | mesh |
Comments
Comment #1
mesh commentedpartly answered. but i dont know how do convert this as an optional formatter.
Comment #2
mrfelton commentedI'm looking to do swappable rollover images too (make an image greyscale when hovered over). Is this possible with the patch at #1?
Comment #3
rjbrown99 commentedThanks 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.
Comment #4
mesh commented@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 :)
Comment #5
mesh commented@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?
Comment #6
rjbrown99 commentedI'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.
Comment #7
David Hernández commentedSubscribing.
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 :(
Comment #8
mrfelton commentedWouldn'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..?
Comment #9
David Hernández commented@mrfelton, I think your approach is the best solution if we don't want to show a huge list of options.
Comment #10
mesh commentedHow 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.
Comment #11
rjbrown99 commentedPatches welcome. I don't need this feature myself, but I'm happy to review and/or evaluate enhancements. Until then I'm flagging postponed.
Comment #12
sballinger26 commentedhey 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
Comment #13
mesh commented@sballinger26: good to know that the code helped..
Comment #14
rjbrown99 commented#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.
Comment #15
rjbrown99 commentedComment #16
mesh commented@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).
Comment #17
mesh commentedi have set up a demo site here - http://sandbox.openmap.inThe demo site is not active any more, sorry.Comment #18
rjbrown99 commented#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.
Comment #19
uno commented#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 :)
Comment #20
rjbrown99 commentedComment #21
mesh commented@rjbrown99: let me know how we can get this thing done further, and what info you need.
Comment #22
rjbrown99 commented#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.
Comment #23
robloachMight actually be useful... Definitely doable in Drupal 7 with the Display Settings.
Comment #24
robloachFixed in Drupal 7 at #1150486: Make use of theme('image_style') instead of just theme('image').
Comment #25
naeluh commented@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
Comment #26
naeluh commented@rjbrown99
Hi do you have any insite as to why this would be happening ?
thanks
Nick
Comment #27
naeluh commentedComment #28
rjbrown99 commentedPlease 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.
Comment #29
naeluh commentedMy 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
Comment #30
open social commented4# Works nice!
Comment #31
sarhansg commentedWill 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.
Comment #32
open social commentedHave you tried the patch in #4?
Comment #33
sarhansg commentedNo, I have not as my site is in a live/production environment. Might set up a test site to play around with it.
Comment #34
Andre-BDrupal 6 support ended on February 24th 2016. https://www.drupal.org/drupal-6-eol