The following patch restores the old behaviour (at least in my use case):

--- Menu.js.orig        Mon Dec 15 17:39:48 2014
+++ Menu.js     Fri May 22 13:15:39 2015
@@ -640,7 +640,7 @@
          // Target is a widget when a menu opens with help of an other widget
          // like a button. In that case getBounds returns an object with top 
and left.
          var target = this._placementTarget;
-        var height = target.top || target.getBounds().top;
+        var height = rootHeight - (target.top || target.getBounds().top);
          this._assertSlideBar(function() {
            this.setHeight(height);
          });


I am not sure if this is the correct fix.

Also, I am wondering what setMaxHeigth() on a sub menu is supposed to do. At
least in the case where the list of menu buttons is higher than maxHeight()
there are no slide controls.

Cheers,
Fritz

On Fri, 22 May 2015, Fritz Zaucker wrote:

> Hi,
>
> http://tinyurl.com/ph6f9zk
>
> (try n=5 and n=40)
>
> If a sub menu is "higher" than the available desktop space then it is
> displayed at the very top of the screen and the sub menu buttons are not
> shown at all.
>
> This was still working in 3.5.1.
>
> There was a change in 4.x in __updateSlideBar() ... ( else if (top === 0)
> branch added). Removing that branch leads to a too large sub menu not
> showing the sldie bar controls, however.
>
> Unfortunately, I am not sure what the correct fix is. Perhaps something to
> do with the mouse/pointer change in 4.x?
>
> A workaround is to setMaxHeight() on the sub menu, but this is not very nice
> ...
>
> Cheers,
> Fritz
>
>

-- 
Oetiker+Partner AG              tel: +41 62 775 9903 (direct)
Fritz Zaucker                        +41 62 775 9900 (switch board)
Aarweg 15                            +41 79 675 0630 (mobile)
CH-4600 Olten                   fax: +41 62 775 9905
Schweiz                         web: www.oetiker.ch

------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud 
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
qooxdoo-devel mailing list
qooxdoo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to