From 50c08401468f6818a159e07a58697272be072811 Mon Sep 17 00:00:00 2001
From: Thomas Lamprecht <t.lamprecht@proxmox.com>
Date: Wed, 12 Jan 2022 12:36:36 +0100
Subject: [PATCH] ui: sys config: merge webauthn and general options into one
 tab

To much wasted space else.
Also rename "Options" to "Others", while it's not _that_ much better
it's slightly more intuitive than config -> options (which has some
redundancy)...

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
---
 www/SystemConfiguration.js | 36 +++++++-----------------------------
 1 file changed, 7 insertions(+), 29 deletions(-)

diff --git a/www/SystemConfiguration.js b/www/SystemConfiguration.js
index ac24bff95..cb07f1141 100644
--- a/www/SystemConfiguration.js
+++ b/www/SystemConfiguration.js
@@ -45,30 +45,9 @@ Ext.define('PBS.SystemConfiguration', {
 	    ],
 	},
 	{
-	    title: gettext('Authentication'),
-	    itemId: 'authentication',
-	    xtype: 'panel',
-	    layout: {
-		type: 'vbox',
-		align: 'stretch',
-		multi: true,
-	    },
-	    defaults: {
-		collapsible: true,
-		animCollapse: false,
-		margin: '10 10 0 10',
-	    },
-	    items: [
-		{
-		    title: gettext('Webauthn'),
-		    xtype: 'pbsWebauthnConfigView',
-		},
-	    ],
-	},
-	{
-	    title: gettext('Options'),
-	    itemId: 'options',
 	    xtype: 'panel',
+	    title: gettext('Other'),
+	    itemId: 'other-options',
 	    layout: {
 		type: 'vbox',
 		align: 'stretch',
@@ -84,6 +63,10 @@ Ext.define('PBS.SystemConfiguration', {
 		    title: gettext('General'),
 		    xtype: 'pbsNodeOptionView',
 		},
+		{
+		    title: gettext('WebAuthn TFA'),
+		    xtype: 'pbsWebauthnConfigView',
+		},
 	    ],
 	},
     ],
@@ -98,12 +81,7 @@ Ext.define('PBS.SystemConfiguration', {
 	    item.relayEvents(networktime, ['activate', 'deactivate', 'destroy']);
 	});
 
-	let authentication = me.getComponent('authentication');
-	Ext.Array.forEach(authentication.query(), function(item) {
-	    item.relayEvents(authentication, ['activate', 'deactivate', 'destroy']);
-	});
-
-	let options = me.getComponent('options');
+	let options = me.getComponent('other-options');
 	Ext.Array.forEach(options.query(), function(item) {
 	    item.relayEvents(options, ['activate', 'deactivate', 'destroy']);
 	});