Porting hovertips to 6.x

Yes, eventually. Patches are welcome, if anyone wants to help. It's a small module and may not need much to upgrade.

Hi,

I have applied this patch, and now hovertip seems to be working fine in my Drupal 6.1 installation.

Cheers,
thomas

Dave, is there any chance you could commit this patch?

(After branching CVS appropriately, of course)

Hmm. Doesn't seem to work quite right.

http://www.housingaffordabilityaustralia.com/content/hovertips-test

I'm using the chameleon theme that comes with D6. In firefox, the hovertip comes up any time I'm over that window, rather than just when I hover over the appropriate text. Maybe I'm doing something wrong too, which is why I'm posting the link here.

Ok, maybe it does, and I'm wrong. I finally got the Glossary and Hovertips working together here, and they seem to work properly :).

...so I guess what I'm saying is, the patch is fine, but the documentation could probably use some work :).

Ok, I've found a problem -- hovertips are only working if I'm logged in as Administrator, not anonymous. That's no reason not to create a 6.x dev branch, though :).

http://drupal.org/node/114774#upgraded_to_jquery_12

@NancyDru: Sorry, that wasn't directed at you, but more at someone who wants to get in and give this problem a shot.

I think the new version of jquery could make hovertip.js shorter (because I think jquery provides some of the coordinate nastiness in there now) although I'm not certain about that.

I've been sluggish to check in an Drupal 6 changes because personally I have not yet upgraded any of my sites. I will get to this, though! I appreciate the help.

I resolved the issue with non-admin users being able to see the tips. The problem was in hovertip_menu, in that apparently access arguments can't simply be array(TRUE). I set it to array('access content').

I also reviewed the js, and it didn't look like it needed any updating.

They can be, but I may not have done it properly.

I assume it can be done, I just couldn't figure out how to do it via the Drupal API.

I suppose one could simple change the "access callback" to "is_array" and pass in "array(1)", but this seems hacky at best.

@Dave: Can we at least have a dev branch for this? If you need help with the release system (including CVS), I can help with that (I assume you don't need this, but since there's been no 1.x version, I thought I'd offer).

@jhedstrom: what is this patch against?

Ok, here's mine against current HEAD, which Dave is updating for the first time since last year. Hopefully he'll be willing to branch for 6.x :).

Forgot to change status; sorry. Nancy implied that she'd done as much as she could on it, so I'm reassigning it to no-one.

Thanks everyone for your work on this. And also for your patience.

Sadly, I'm stuck in the Drupal 5.x world for every site I'm working on. So setting up my environment for 6.x has been something of a hurdle. I don't want to delay you any further so I've checked in the patch - without testing it. Something I'm loathe to do, but just resurrect this issue if I've blundered.

http://drupal.org/node/259042