From d4b07820bed2576b13cf02cee1b3c1336b45e0cc Mon Sep 17 00:00:00 2001 From: Denis Flaven Date: Wed, 9 Jun 2010 17:17:39 +0000 Subject: [PATCH] - Fixed the positioning of the iTop popup menus (Trac #316) SVN:trunk[451] --- js/jquery.popupmenu.js | 2 +- js/jquery.positionBy.js | 16 ++++++++++++++++ 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/js/jquery.popupmenu.js b/js/jquery.popupmenu.js index fe27e70f7..5699809c9 100644 --- a/js/jquery.popupmenu.js +++ b/js/jquery.popupmenu.js @@ -35,7 +35,7 @@ jQuery.fn.popupmenu = function () // We really clicked in a different menu, let's open it popupmenu.bgiframe(); popupmenu.positionBy({ target: $(this), - targetPos: 2, + targetPos: 4, elementPos: 0, hideAfterPosition: true }); diff --git a/js/jquery.positionBy.js b/js/jquery.positionBy.js index 23dd1148e..9857ffcd1 100644 --- a/js/jquery.positionBy.js +++ b/js/jquery.positionBy.js @@ -9,6 +9,10 @@ * * Built upon jQuery 1.2.2 (http://jquery.com) * This also requires the jQuery dimensions plugin + * + * + * @@ Modified by dflaven for handling the positioning of the iTop popup menus @@ + * */ (function($){ /** @@ -205,6 +209,11 @@ position[15]= new Range(tLeft, tTop, tLeft + eWidth, tTop + eHeight); next[15] = [14,0,9]; + // @@ Added by dflaven + position[16]= new Range(tRight - eWidth, tBottom, tRight, tBottom + eHeight); + next[16] = [3,10]; + // @@End of modification + if ( args.positions !== null ) { var pos = args.positions[0]; } else if ( args.targetPos != null && args.elementPos != null ) { @@ -229,6 +238,13 @@ pos[3][1] = 5; pos[3][2] = 6; pos[3][3] = 14; + // @@ Added by dflaven + pos[4] = []; + pos[4][0] = 16; + pos[4][1] = 16; + pos[4][2] = 16; + pos[4][3] = 16; + // @@ End of modification var pos = pos[args.targetPos][args.elementPos]; }