From dad14ab0e0b232bf9173c5d6d64bff0359c3d343 Mon Sep 17 00:00:00 2001 From: Thomas Lamprecht Date: Fri, 29 Jun 2018 13:05:02 +0200 Subject: [PATCH] ui/storage: cifs: reoder fields username, password and domain name are all related, they are non-pve options related to accessing the CIFS storage and have all influence on the pveCIFSScan component. So group at kleast user and password together. But keep domain on the right for the sake of balance. Move the content type selector to the right, as the RBD and other panels do. Signed-off-by: Thomas Lamprecht --- www/manager6/storage/CIFSEdit.js | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/www/manager6/storage/CIFSEdit.js b/www/manager6/storage/CIFSEdit.js index 2be08c7a..8a3ddb1f 100644 --- a/www/manager6/storage/CIFSEdit.js +++ b/www/manager6/storage/CIFSEdit.js @@ -142,20 +142,13 @@ Ext.define('PVE.storage.CIFSInputPanel', { } } }, + passwordfield, { xtype: me.isCreate ? 'pveCIFSScan' : 'displayfield', name: 'share', value: '', fieldLabel: 'Share', allowBlank: false - }, - { - xtype: 'pveContentTypeSelector', - name: 'content', - value: 'images', - multiSelect: true, - fieldLabel: gettext('Content'), - allowBlank: false } ]; @@ -170,7 +163,14 @@ Ext.define('PVE.storage.CIFSInputPanel', { value: me.isCreate ? '1' : undefined, allowBlank: false }, - passwordfield, + { + xtype: 'pveContentTypeSelector', + name: 'content', + value: 'images', + multiSelect: true, + fieldLabel: gettext('Content'), + allowBlank: false + }, { xtype: me.isCreate ? 'textfield' : 'displayfield', name: 'domain',