[pve-devel] [PATCH manager v2 3/3] Move Max Backups field below the Content Type Selector

Emmanuel Kasper e.kasper at proxmox.com
Tue Oct 31 15:52:11 CET 2017


Since the Max Backups field is now depending of having the Content Type
set to 'backup', it makes sense to group these two fields.

Signed-off-by: Emmanuel Kasper <e.kasper at proxmox.com>
---
 www/manager6/storage/DirEdit.js       | 22 +++++++++++-----------
 www/manager6/storage/GlusterFsEdit.js | 22 +++++++++++-----------
 www/manager6/storage/NFSEdit.js       | 22 +++++++++++-----------
 3 files changed, 33 insertions(+), 33 deletions(-)

diff --git a/www/manager6/storage/DirEdit.js b/www/manager6/storage/DirEdit.js
index 4e196431..8a8d6d4c 100644
--- a/www/manager6/storage/DirEdit.js
+++ b/www/manager6/storage/DirEdit.js
@@ -44,6 +44,17 @@ Ext.define('PVE.storage.DirInputPanel', {
 		multiSelect: true,
 		fieldLabel: gettext('Content'),
 		allowBlank: false
+	    },
+	    {
+		xtype: 'pveIntegerField',
+		fieldLabel: gettext('Max Backups'),
+		disabled: true,
+		name: 'maxfiles',
+		reference: 'maxfiles',
+		minValue: 0,
+		maxValue: 365,
+		value: me.isCreate ? '1' : undefined,
+		allowBlank: false
 	    }
 	];
 
@@ -60,17 +71,6 @@ Ext.define('PVE.storage.DirInputPanel', {
 		name: 'shared',
 		uncheckedValue: 0,
 		fieldLabel: gettext('Shared')
-	    },
-	    {
-		xtype: 'pveIntegerField',
-		fieldLabel: gettext('Max Backups'),
-		disabled: true,
-		name: 'maxfiles',
-		reference: 'maxfiles',
-		minValue: 0,
-		maxValue: 365,
-		value: me.isCreate ? '1' : undefined,
-		allowBlank: false
 	    }
 	];
 
diff --git a/www/manager6/storage/GlusterFsEdit.js b/www/manager6/storage/GlusterFsEdit.js
index d861c0b8..9ab97761 100644
--- a/www/manager6/storage/GlusterFsEdit.js
+++ b/www/manager6/storage/GlusterFsEdit.js
@@ -127,6 +127,17 @@ Ext.define('PVE.storage.GlusterFsInputPanel', {
 		multiSelect: true,
 		fieldLabel: gettext('Content'),
 		allowBlank: false
+	    },
+	    {
+		xtype: 'pveIntegerField',
+		fieldLabel: gettext('Max Backups'),
+		disabled: true,
+		name: 'maxfiles',
+		reference: 'maxfiles',
+		minValue: 0,
+		maxValue: 365,
+		value: me.isCreate ? '1' : undefined,
+		allowBlank: false
 	    }
 	];
 
@@ -146,17 +157,6 @@ Ext.define('PVE.storage.GlusterFsInputPanel', {
 		checked: true,
 		uncheckedValue: 0,
 		fieldLabel: gettext('Enable')
-	    },
-	    {
-		xtype: 'pveIntegerField',
-		fieldLabel: gettext('Max Backups'),
-		disabled: true,
-		name: 'maxfiles',
-		reference: 'maxfiles',
-		minValue: 0,
-		maxValue: 365,
-		value: me.isCreate ? '1' : undefined,
-		allowBlank: false
 	    }
 	];
 
diff --git a/www/manager6/storage/NFSEdit.js b/www/manager6/storage/NFSEdit.js
index 728a3ffd..526ce1db 100644
--- a/www/manager6/storage/NFSEdit.js
+++ b/www/manager6/storage/NFSEdit.js
@@ -122,6 +122,17 @@ Ext.define('PVE.storage.NFSInputPanel', {
 		multiSelect: true,
 		fieldLabel: gettext('Content'),
 		allowBlank: false
+	    },
+	    {
+		xtype: 'pveIntegerField',
+		fieldLabel: gettext('Max Backups'),
+		disabled: true,
+		name: 'maxfiles',
+		reference: 'maxfiles',
+		minValue: 0,
+		maxValue: 365,
+		value: me.isCreate ? '1' : undefined,
+		allowBlank: false
 	    }
 	];
 
@@ -141,17 +152,6 @@ Ext.define('PVE.storage.NFSInputPanel', {
 		checked: true,
 		uncheckedValue: 0,
 		fieldLabel: gettext('Enable')
-	    },
-	    {
-		xtype: 'pveIntegerField',
-		fieldLabel: gettext('Max Backups'),
-		disabled: true,
-		name: 'maxfiles',
-		reference: 'maxfiles',
-		minValue: 0,
-		maxValue: 365,
-		value: me.isCreate ? '1' : undefined,
-		allowBlank: false
 	    }
 	];
 
-- 
2.11.0




More information about the pve-devel mailing list