Placeholder for Drupal 7 port. This will eventually be done, and in the meantime patches are welcome in case someone is using the module and wants to start a -dev version for 7.

Comments

wilgrace’s picture

+1

d.olaresko’s picture

Rewrote this module. Now works on D7.

d.olaresko’s picture

StatusFileSize
new17.95 KB

oops. wrong attachment.

rjbrown99’s picture

Hi and thank you very much for posting this. Would you mind re-posting without the copyright notices throughout? Thanks.

d.olaresko’s picture

StatusFileSize
new9.83 KB

Yeap. Sure. See attachment. Also added small fixes for multivalues fields =)

rjbrown99’s picture

Thanks. So to be clear, this is not just a straight port to 7 but includes additional fixes? What I would want to do with a 7.x tree is to keep both the 7.x and 6.x codebases in step as much as possible. Can you highlight what changed (aside from the ports - more interested in the 'multivalue fixes')?

I guess depending on how it looks I could diff it the two trees, but I may not spot all of the changes depending on how hard the 7.x port was. I'm not using 7 at all yet so I'm working entirely off of your code.

Also, you may want to check out the 6.x commit from this weekend, it should fix the javascript issue of images being hovered offscreen.

Thanks again for the port, very helpful.

rjbrown99’s picture

#5 - I now have a D7 site that I can test with. Can you shed some light on your "small fixes for multivalue fields"? I'd like to start a 7.x branch but I'd also like to backport any enhancements to the D6 version.

d.olaresko’s picture

Yeap. If you try to use hover preview for field that has more than one image (see "NUMBER OF VALUES" parameter at the field settings page) it will works theme each image separately =)

drupa11y’s picture

THANKS - it´s exactly what I was looking for.

robloach’s picture

Status: Active » Needs review
Issue tags: +D7CX, +hover_preview
StatusFileSize
new20.69 KB
new16.33 KB
new2.14 KB

This is pretty much a complete rewrite. There's so much more nicer stuff we can do with the Fields API, having a straight port just doesn't really make sense. This gets rid of the multiple field formatters, and just uses the field formatter settings to provide the Hover Preview styles instead.

Attached is a patch, the entire module, and a screenshot of it using the formatter settings. #1025796: Rename path to uri in image functions would also be nice to have.

Thanks!

robloach’s picture

Status: Fixed » Closed (fixed)
Issue tags: -D7CX, -hover_preview

Automatically closed -- issue fixed for 2 weeks with no activity.