[pve-devel] [PATCH manager 7/8] ext6migrate: fix HDEdit, HDMove, HDResize

Dominik Csapak d.csapak at proxmox.com
Fri Mar 18 14:36:10 CET 2016


the StorageSelector fires the change event one time before data is there,
this patch catches that

also make the hdresize window wider because of the bigger font

Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
 www/manager6/qemu/HDEdit.js   | 3 +++
 www/manager6/qemu/HDMove.js   | 3 +++
 www/manager6/qemu/HDResize.js | 2 +-
 3 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/www/manager6/qemu/HDEdit.js b/www/manager6/qemu/HDEdit.js
index 37cd8c7..333c658 100644
--- a/www/manager6/qemu/HDEdit.js
+++ b/www/manager6/qemu/HDEdit.js
@@ -169,6 +169,9 @@ Ext.define('PVE.qemu.HDInputPanel', {
 		allowBlank: false,
 		listeners: {
 		    change: function(f, value) {
+			if (!value) { // initial store loading fires an unwanted 'change'
+			    return;
+			}
 			var rec = f.store.getById(value);
 			if (rec.data.type === 'iscsi') {
 			    me.hdfilesel.setStorage(value);
diff --git a/www/manager6/qemu/HDMove.js b/www/manager6/qemu/HDMove.js
index 6144060..745c791 100644
--- a/www/manager6/qemu/HDMove.js
+++ b/www/manager6/qemu/HDMove.js
@@ -70,6 +70,9 @@ Ext.define('PVE.window.HDMove', {
                 hidden: false,
                 listeners: {
                     change: function(f, value) {
+			if (!value) { // initial store loading fires an unwanted 'change
+			    return;
+			}
                         var rec = f.store.getById(value);
 			if (rec.data.type === 'iscsi' ||
 			    rec.data.type === 'lvm' ||
diff --git a/www/manager6/qemu/HDResize.js b/www/manager6/qemu/HDResize.js
index 70ec6bd..0e50ff8 100644
--- a/www/manager6/qemu/HDResize.js
+++ b/www/manager6/qemu/HDResize.js
@@ -59,7 +59,7 @@ Ext.define('PVE.window.HDResize', {
 	    bodyPadding: 10,
 	    border: false,
 	    fieldDefaults: {
-		labelWidth: 120,
+		labelWidth: 140,
 		anchor: '100%'
 	    },
 	    items: items
-- 
2.1.4





More information about the pve-devel mailing list