MDL-37566 AJAX Only set max-height for chooser dialogues
authorAndrew Robert Nicols <andrew.nicols@luns.net.uk>
Mon, 28 Jan 2013 23:52:48 +0000 (23:52 +0000)
committerAndrew Robert Nicols <andrew.nicols@lancaster.ac.uk>
Fri, 1 Feb 2013 11:21:47 +0000 (11:21 +0000)
If we set height as well as max-height then there is a potential for
whitespace to be shown at the bottom of the list of choices.

This commit stops removing the height option so that we only rely upon
max-height for dialogue heigh calculation.

lib/yui/chooserdialogue/chooserdialogue.js

index cb7d54a..897f0c5 100644 (file)
@@ -239,8 +239,7 @@ YUI.add('moodle-core-chooserdialogue', function(Y) {
             // new max-height
             totalheight = newheight;
             newheight = newheight - (15 + 15 + 40 + 40);
-            dialogue.setStyle('max-height', newheight + 'px');
-            dialogue.setStyle('height', newheight + 'px');
+            dialogue.setStyle('maxHeight', newheight + 'px');
 
             // Re-calculate the location now that we've changed the size
             dialoguetop = Math.max(12, ((winheight - totalheight) / 2)) + offsettop;