Ok this is really, really minor but still.

The new edit_view hovertip that comes up in panels is very, very handy. However, in any panel pane where it's active, when you hover over any other link, it stops the url showing up in the status bar (on FF anyway).

CommentFileSizeAuthor
#10 panels_css.patch1.31 KByched
#3 panels.js_.patch935 bytesyched

Comments

yched’s picture

Yep, noticed that too (on FF2/win). That's somewhat annoying :-)

merlinofchaos’s picture

I've committed a check so that the javascript only gets included if you actually have permission to view the admin links, so at least it won't affect most users. But I'm not sure how to fix this for other users.

It seems that the jquery .hover() method is eating the browser bit that displays the link in the status bar, and I'm not sure what the right way to restore it is.

yched’s picture

Status: Active » Needs review
StatusFileSize
new935 bytes

Attached patch uses .mouseover / .mouseout insetad of .hover, and seems to do the trick.

merlinofchaos’s picture

Status: Needs review » Active

I've committed this but it doesn't actually fix it for me in FF2.

yched’s picture

Ah. Hmm. I'm now away from my panels2 dev site. Will try harder tomorrow...

(I hate to ask : did you clear your browser's cache ? ;-) )

merlinofchaos’s picture

Yup.

Also added console.log messages to ensure I was actually getting fresh .js.

yched’s picture

Well, at least I can confirm it works for me, both on FF2 and IE7.

catch’s picture

hmm, not working for me either.

merlinofchaos’s picture

I am thinking that I may have to go with an alternative solution where we use CSS hovers in browsers that don't suck (Firefox, Opera, Safari) and only use the javascript method for browsers that do (IE). It seems that this is possibly causing other problems; I got a report that something in Panels was interfering with the fivestar gadget, and I bet it's also the hover.

yched’s picture

StatusFileSize
new1.31 KB

I got a report that something in Panels was interfering with the fivestar gadget
Yep, just spent two hours on it :-) it's this issue : http://drupal.org/node/207896.

It's a css conflict actually ("div.panel-pane div.hover" is a too broad selector)
Attached patch fixes it.

merlinofchaos’s picture

I applied the .css patch and then reverted to using .hover() because...hover() works on IE and the mouseover() mouseout() stuff doesn't.

Working on a implementing the css hovers and then only doing the hover() method when it's IE now.

merlinofchaos’s picture

Status: Active » Fixed

Ok, got that working. Let me know if there continue to be problems.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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