Vertical offset with supposition plugin [#1369718]

I tried to use the supposition plugin because I have some very large submenues. But I get a vertical offset of the submenue (see screenshot). I used the space [blue] style.
I know the plugin is beta, but it would be very useful. So any possibility to get this fixed?
Thanks for your help.
grossmann-mcs

Comment	File	Size	Author
#7	Snímek obrazovky 2016-01-06 v 14.34.27.PNG	34.56 KB	Honza Pobořil
	superfish supposition vertical offset.jpg	68.74 KB	grossmann

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

mehrpadin CreditAttribution: mehrpadin commented 23 December 2011 at 19:21

Servus,

I see, it's that the child menu - like all the other child menus - has top: 0;, I guess bottom: 0; can somehow help us here, but need to test it; will add it to my to-do list so it'll be diffidently fixed in the next version, vielen dank :)

Comment #2

grossmann CreditAttribution: grossmann commented 10 January 2012 at 14:20

Hello,
thanks for your hint here. I tried around and found a solution - see comments (sorry I tagged this as php code which it isn't, it's javascript, but it gives a nicer syntax highlighting).

        var windowHeight = $w.height(),
        offsetTop = $u.offset().top,
        menuParentHeight = $u.parent().outerHeight(true),
        menuHeight = $u.height(),
        menuPadding = $(this).css("paddingBottom"), // get paddingBottom for positioning
        baseline = windowHeight + _offset('y');
        var expandUp = ((offsetTop + menuHeight > baseline) && (offsetTop > menuHeight));
        if (expandUp) {
          $u.css('bottom', '-' + menuPadding); // substracts the paddingBottom for alignment
          // $u.css('bottom', menuParentHeight + 'px');
          $u.css('top', 'auto');
        }

As you see I defined bottom not with menuParentHeight but instead I get the paddingBottom value and substract it from bottom to align it with the parent.
Maybe this is a help for your solution. I didn't test it with other setting of superfish so this may not work as a general solution.
I hope this helps.
Bitte schön ;-)
grossmann-mcs

Comment #3

mehrpadin CreditAttribution: mehrpadin commented 22 October 2013 at 09:16

Status:

Active

» Fixed

Oh forgot this issue! fixed some time ago (development release with the master branch of the library)

Comment #4

5 November 2013 at 09:20

Status:

Fixed

» Closed (fixed)

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

Comment #5

mibfire CreditAttribution: mibfire commented 5 January 2015 at 18:40

Issue summary:	View changes
Priority:	Normal	» Major
Status:	Closed (fixed)	» Active

$u.css('top', 'auto'); wont be removed after mouseout, so the submenu will be visible, instead of hidden with top: -99999em;!!!

Comment #6

mibfire CreditAttribution: mibfire commented 5 January 2015 at 20:26

in superfish.js

This

    hideSuperfishUl : function(){
      var o = sf.op,
        not = (o.retainPath===true) ? o.$path : '';
      o.retainPath = false;
      var $ul = $(['li.',o.hoverClass].join(''),this).add(this).not(not).removeClass(o.hoverClass)
          .find('>ul').addClass('sf-hidden');
          
      o.onHide.call($ul);
      return this;
    },

should be

  $.fn.extend({
    hideSuperfishUl : function(){
      var o = sf.op,
        not = (o.retainPath===true) ? o.$path : '';
      o.retainPath = false;
      var $ul = $(['li.',o.hoverClass].join(''),this).add(this).not(not).removeClass(o.hoverClass)
          .find('>ul').addClass('sf-hidden').css('top', '');

      o.onHide.call($ul);
      return this;
    },