[pve-devel] [PATCH manager] rename enableAfterLoad to enableLoadMask

Dominik Csapak d.csapak at proxmox.com
Fri Apr 22 11:47:10 CEST 2016


also add a explaining comment

Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
 www/manager6/form/ComboGrid.js | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/www/manager6/form/ComboGrid.js b/www/manager6/form/ComboGrid.js
index 33f3059..1654e6f 100644
--- a/www/manager6/form/ComboGrid.js
+++ b/www/manager6/form/ComboGrid.js
@@ -221,7 +221,7 @@ Ext.define('PVE.form.ComboGrid', {
 	//
 	// we save the minheight to reset it after the load
 	picker.on('show', function() {
-	    if (me.enableAfterLoad) {
+	    if (me.enableLoadMask) {
 		me.savedMinHeight = picker.getMinHeight();
 		picker.setMinHeight(100);
 	    }
@@ -257,7 +257,7 @@ Ext.define('PVE.form.ComboGrid', {
 
 	me.mon(me.store, 'beforeload', function() {
 	    if (!me.isDisabled()) {
-		me.enableAfterLoad = true;
+		me.enableLoadMask = true;
 	    }
 	});
 
@@ -266,8 +266,13 @@ Ext.define('PVE.form.ComboGrid', {
 	    if (success) {
 		me.clearInvalid();
 
-		if (me.enableAfterLoad) {
-		    delete me.enableAfterLoad;
+		if (me.enableLoadMask) {
+		    delete me.enableLoadMask;
+
+		    // if the picker exists,
+		    // we reset its minheight to the saved var/0
+		    // we have to update the layout, otherwise the height
+		    // gets not recalculated
 		    if (me.picker) {
 			me.picker.setMinHeight((me.savedMinHeight)?me.savedMinHeight:0);
 			delete me.savedMinHeight;
-- 
2.1.4





More information about the pve-devel mailing list