Chirag Patel (OpenERP) has proposed merging 
lp:~openerp-dev/openerp-web/6.1-opw-572769-cpa into lp:openerp-web/6.1.

Requested reviews:
  OpenERP Core Team (openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-572769-cpa/+merge/100083

Hello,

Fixed increase the height of dialog, the internal content height do not 
increase.

Demo:
1) Decrease the height of browser.
2) Open any dialog.
3) Increase the height of browser.
4) Increase the height of dialog.

Observed: dialog has enough height for internal content but scrollbar not 
removed.
Expected: Internal content pane must be resize with dialog box height.

Thanks.

-- 
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-572769-cpa/+merge/100083
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/6.1-opw-572769-cpa.
=== modified file 'addons/web/static/src/js/chrome.js'
--- addons/web/static/src/js/chrome.js	2012-03-28 15:00:31 +0000
+++ addons/web/static/src/js/chrome.js	2012-03-30 08:45:26 +0000
@@ -141,6 +141,11 @@
     },
     on_resized: function() {
         //openerp.log("Dialog resized to %d x %d", this.$element.width(), this.$element.height());
+        var $dialog = this.$element.parent(".ui-dialog");
+        var btnpane_height = $dialog.find(".ui-dialog-buttonpane").height();
+        var content_height = ($dialog.height() - btnpane_height - 62) + "px";
+        
+        this.$element.css({ 'max-height':content_height, 'height':content_height});
     },
     stop: function () {
         // Destroy widget

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to