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.
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | hover_preview7.tar.gz | 2.14 KB | robloach |
| #10 | 800672.patch | 16.33 KB | robloach |
| #10 | Hover Preview Field Formatter Settings Screenshot.png | 20.69 KB | robloach |
| #5 | hover_preview.zip | 9.83 KB | d.olaresko |
| #3 | d7port.patch | 17.95 KB | d.olaresko |
Comments
Comment #1
wilgrace commented+1
Comment #2
d.olaresko commentedRewrote this module. Now works on D7.
Comment #3
d.olaresko commentedoops. wrong attachment.
Comment #4
rjbrown99 commentedHi and thank you very much for posting this. Would you mind re-posting without the copyright notices throughout? Thanks.
Comment #5
d.olaresko commentedYeap. Sure. See attachment. Also added small fixes for multivalues fields =)
Comment #6
rjbrown99 commentedThanks. 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.
Comment #7
rjbrown99 commented#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.
Comment #8
d.olaresko commentedYeap. 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 =)
Comment #9
drupa11y commentedTHANKS - it´s exactly what I was looking for.
Comment #10
robloachThis 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!
Comment #11
robloachThanks! http://drupalcode.org/project/hover_preview.git/commit/55f5647