comestic : switch proto and interface in forms

for readability, better to have

macro
proto
src port
dest port

in same column

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

www/manager/grid/FirewallRules.js

@@ -70,18 +70,27 @@ Ext.define('PVE.FirewallRulePanel', {
 		data: [['ACCEPT', 'ACCEPT'], ['DROP', 'DROP'], ['REJECT', 'REJECT']],
 		fieldLabel: gettext('Action'),
 		allowBlank: false
-	    },
+	    }
-	    {
+        ];
-		xtype: 'pveKVComboBox',
+
-		name: 'proto',
+	if (me.allow_iface) {
-		value: '',
+	    me.column1.push({
+		xtype: 'pvetextfield',
+		name: 'iface',
 		deleteEmpty: !me.create,
-		emptyText: 'any',
+		value: '',
-		editable: true,
+		fieldLabel: gettext('Interface')
-		data: [['tcp', 'TCP'], ['udp', 'UDP'], ['icmp', 'ICMP']],
+	    });
-		fieldLabel: gettext('Protocol'),
+	} else {
-		allowBlank: true
+	    me.column1.push({
-	    },
+		xtype: 'displayfield',
+		fieldLabel: '',
+		height: 22, // hack: set same height as text fields
+		value: ''
+	    });
+	}
+
+	me.column1.push([
 	    {
 		xtype: 'displayfield',
 		fieldLabel: '',
@@ -106,7 +115,7 @@ Ext.define('PVE.FirewallRulePanel', {
 		value: '',
 		fieldLabel: gettext('Destination')
 	    }
-	];
+	]);
 
 	me.column2 = [
 	    {
@@ -124,27 +133,18 @@ Ext.define('PVE.FirewallRulePanel', {
 		deleteEmpty: !me.create,
 		fieldLabel: gettext('Macro'),
 		allowBlank: true
-	    }
+	    },
-	];
+	    {
-
+		xtype: 'pveKVComboBox',
-	if (me.allow_iface) {
+		name: 'proto',
-	    me.column2.push({
-		xtype: 'pvetextfield',
-		name: 'iface',
-		deleteEmpty: !me.create,
 		value: '',
-		fieldLabel: gettext('Interface')
+		deleteEmpty: !me.create,
-	    });
+		emptyText: 'any',
-	} else {
+		editable: true,
-	    me.column2.push({
+		data: [['tcp', 'TCP'], ['udp', 'UDP'], ['icmp', 'ICMP']],
-		xtype: 'displayfield',
+		fieldLabel: gettext('Protocol'),
-		fieldLabel: '',
+		allowBlank: true
-		height: 22, // hack: set same height as text fields
+	    },
-		value: ''
-	    });
-	}
-
-	me.column2.push([
 	    {
 		xtype: 'displayfield',
 		fieldLabel: '',
@@ -164,7 +164,7 @@ Ext.define('PVE.FirewallRulePanel', {
 		value: '',
 		fieldLabel: gettext('Dest. port')
 	    }
-	]);
+	];
 	
 	me.columnB = [
 	    {