Hi there,

Taking the following list as example:

  • Item 1
    • Item 1.1
    • Item 1.2
  • Item 2
  • Item 3
    • Item 3.1
    • Item 3.2

I have noticed that when you hover Item 2, which doesn't have children, the fired event doesn't hide any of the other children list. I've dig a little bit in the code and I found the following lines in superfish.js:

$('li:has(ul)',this)[($.fn.hoverIntent && !o.disableHI) ? 'hoverIntent' : 'hover'](over,out).each(function() {
if (o.autoArrows) addArrow( $('>a:first-child',this) );
})
.not('.'+c.bcClass)
.hideSuperfishUl();

Removing the 'has(ul)' from the selector makes the trick but I'm wondering if this is the right way to solve this problem or if this is not even a bug.

Can you please check this behaviour?

Just in case this is relevant the style that I'm using is 'default' and the menu type is 'NavBar'.

Thank you very much, and nice work.

CommentFileSizeAuthor
#6 navbar.png22.67 KBPoieo
#3 superfish.patch504 bytesaltrugon

Comments

mehrpadin’s picture

mehrpadin commented

Hola Alberto,

I'm afraid this is the standard behaviour of the SF plugin, if you think it needs to be changed you can contact Joel - the developer of SF plugin.

:)

altrugon’s picture

altrugon commented

Thank you Mehrpadin.

altrugon’s picture

altrugon commented
Status: Active » Needs review
StatusFileSize
new superfish.patch504 bytes

I know this is an ugly fix but I haven't been able to get a hold of the author of superfish plugin.

In the meantime here is a patch for this problem, I hope this help you guys.

maxferrario’s picture

maxferrario commented

Nice patch, Alberto: even if this in not "standard behaviour" is a very reasonable one!
I wonder if this could become a configurable option...

mehrpadin’s picture

mehrpadin commented
Status: Needs review » Closed (works as designed)

Update: Most probably the reported behaviour is a result of an "overdose" in the Menu delay option, you should set it to 800 or so and SF will work as expected.

The patch therefore is not recommended, and actually must be avoided as it prevents SF from functioning properly.

Poieo’s picture

Poieo commented
Version: 6.x-1.6 » 7.x-1.x-dev
StatusFileSize
new navbar.png22.67 KB

@mehrapdin - I think you're confused about the original issue. The problem here, related specifically to the navbar setup, is that the active parent's child menu does not hide when another parent menu, without children, is hovered.

The effect, to the end user, is that the hovered parent appears to have the same children as the active parent.

Regardless, it appears to be a plugin issue, not a module issue.

See attached.