D7 ver

yep-- there will definitely be a d7 version I just haven't had a chance to get to it yet.

I'm also eagerly awaiting a version that works with D7 :)

sub..nice module!

Subscribing - Great to know ;-)

subscribe

Any timeline for D7?

Subscribing - changed status to match #7

(tired of subscribing ? http://drupal.org/node/34496 - http://3281d.com/2009/03/27/death-to-subscribe-comments)

i tried #7 jp_maphilight but it didnt work. I tested with the example on the README.txt.
In the settings page, the "JQUERY MAP HILIGHT PLUGIN STATUS" seems to be ok, "The Map Hilight jQuery plugin is available at: /sites/all/modules/jquery_plugin/jquery.maphilight.min.js"
Any help?

tks

Attached is a patch against the current HEAD upgrading it to Drupal 7.x.

New features:

• Drush support for plugin download: drush maphilight-plugin
• No inline JS, now created as an external file. Supports JS aggregating
• New maphilight features added to config: neverOn, groupBy, and wrapClass
• Config option to not automatically add library to all pages
• Uses Drupal Behaviors, fixes: #1053786: Map Hilight & Ajax not working together
• Libraries API required
• jQuery Plugin module no longer required, but still supported

Run git apply 1030332-D7Upgrade-10-D7.patch in the module root to apply.

Argh. Patch updated (git diff HEAD, not simply git diff) to include drush/jq_maphilight.drush.inc

eosrei-- you rock! I'm still buried by my day job atm, but I hope to get to this before the weekend. I'll try to get a dev committed asap. Thanks so much for the patch.

Updated again ;) Docs for groupBy and wrapClass are wrong at: http://davidlynch.org/js/maphilight/docs/ Both are bool, not string. Corrected.

did a quick patch review-- looks great.

One thing though, the registry was partly rolled back and I'm not sure the files[] directive needs to be included in the .info file in this case.

I was basing some changes on other modules. I agree, the file[] statement is unneeded.

ok-- patch committed (without the files[] directives in .info, and some coder style fixes).

Report any new problems in new issues against the 7.x-1.x branch. The dev release should be available within 12 hours.

Thanks again eosrei!

@eosrei

can you comment on #1308734: Use of jQuery Map Hilight module broken if Drupal files directory is not in www root. (be sure to see the linked issue as well). Since the getcwd() wasn't in my d6 code, I have no idea if you had a specific reason for including it there. The d6 version works fine without it as far as I can tell.

See other issue.