[mms] Make dynamic context menus scrollable if taller than the browse…

…r screen height (Bug #13833). Conflicts: horde/docs/CHANGES horde/package.xml
horde · Feb 3, 2015 · e326a0f · e326a0f
1 parent 202aca9
commit e326a0f
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 3 deletions.
diff --git a/horde/docs/CHANGES b/horde/docs/CHANGES
@@ -12,6 +12,15 @@ v5.3.0-git
       #13200).
 
 
+------
+v5.2.4
+------
+[mms] Make dynamic context menus scrollable if taller than the browser screen
+      height (Bug #13833).
+[jan] Fix horde-import-squirrelmail-prefs script (Bug #13780).
+[jan] Don't instantiate notification system for RPC requests.
+
+
 ------
 v5.2.3
 ------

diff --git a/horde/js/contextsensitive.js b/horde/js/contextsensitive.js
@@ -317,12 +317,19 @@ var ContextSensitive = Class.create({
         eltL = elt.getLayout();
         h = eltL.get('border-box-height');
         w = eltL.get('border-box-width');
-        elt.hide().setStyle({ visibility: 'visible' });
+        elt.hide().setStyle({
+            height: 'auto',
+            visibility: 'visible'
+        });
         v = document.viewport.getDimensions();
 
         // Make sure context window is entirely on screen
-        if ((y + h) > v.height) {
-            y = v.height - h - 2;
+        if ((y + h + 5) > v.height) {
+            y = Math.max(5, v.height - h - 5);
+
+            if (h - 10 > v.height) {
+                elt.setStyle({ height: (v.height - 10) + 'px' });
+            }
         }
 
         if ((x + w) > v.width) {

diff --git a/horde/package.xml b/horde/package.xml
@@ -4102,6 +4102,7 @@
    <date>2014-12-03</date>
    <license uri="http://www.horde.org/licenses/lgpl">LGPL-2</license>
    <notes>
+* [mms] Make dynamic context menus scrollable if taller than the browser screen height (Bug #13833).
 * [jan] Fix horde-import-squirrelmail-prefs script (Bug #13780).
 * [jan] Don&apos;t instantiate notification system for RPC requests.
    </notes>

diff --git a/horde/themes/default/screen.css b/horde/themes/default/screen.css
@@ -1400,6 +1400,7 @@ div.GrowlerNoticeExit {
 .context {
     top: 0;
     left: 0;
+    overflow-y: auto;
 }
 .context a, .context a:visited, .horde-buttonbar li div li a {
     white-space: nowrap;