From: Michael M Slusarz <slusarz@curecanti.org>
Date: Wed, 12 May 2010 17:25:59 +0000 (-0600)
Subject: Fix KNL width positioning
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=8e6c5b6d68e248ad31968d8084ad86fb6817e703;p=horde.git

Fix KNL width positioning
---

diff --git a/horde/js/KeyNavList.js b/horde/js/KeyNavList.js
index ea6d71011..58c4c276c 100644
--- a/horde/js/KeyNavList.js
+++ b/horde/js/KeyNavList.js
@@ -156,22 +156,25 @@ var KeyNavList = Class.create({
     _sizeDiv: function()
     {
         var divL = this.div.getLayout(),
-            dl = divL.get('left'), dt = divL.get('top'),
+            dl = divL.get('left'),
+            dt = divL.get('top'),
             off = this.opts.domParent.cumulativeOffset(),
             v = document.viewport.getDimensions();
 
-        if ((divL.get('border-box-width') + dl + off.left + 10) > v.width) {
-            dl = (v.width - this.div.scrollWidth - off.left - 10);
-            this.div.setStyle({ left: dl + 'px' });
-        }
-
         if ((divL.get('border-box-height') + dt + off.top + 10) > v.height) {
             this.div.setStyle({
                 height: (v.height - dt - off.top - 10) + 'px',
-                width: (this.div.scrollWidth + 10) + 'px'
+                width: (this.div.scrollWidth + 5) + 'px'
             });
         }
 
+        /* Need to do width second - horizontal scrolling might add scroll
+         * bar. */
+        if ((divL.get('border-box-width') + dl + off.left + 5) > v.width) {
+            dl = (v.width - divL.get('border-box-width') - off.left - 5);
+            this.div.setStyle({ left: dl + 'px' });
+        }
+
         if (this.iefix) {
             this.iefix.clonePosition(this.div);
         }