Harmonize reset buttons in "My Settings"

- Rename both buttons to a single word as it is clear where they belong
  to. Use "Reset" instead of "Clear" as you can rather "Reset a saved
  user name" than "Clear a layout".
- Use container instead of panels as they are simpler (e.g. no "border:
  false" necessary) and sufficient.
- Align the buttons at the right for visual appeal.
- Use title case for the user name label for consistency.

Signed-off-by: Dominic Jäger <d.jaeger@proxmox.com>
This commit is contained in:
Dominic Jäger 2019-10-03 11:58:38 +02:00 committed by Thomas Lamprecht
parent e87bc6bee4
commit bd7aef8664

View File

@ -218,46 +218,46 @@ Ext.define('PVE.window.Settings', {
xtype: 'box', xtype: 'box',
autoEl: { tag: 'hr'} autoEl: { tag: 'hr'}
}, },
{
xtype: 'container',
layout: 'hbox',
items: [
{ {
xtype: 'displayfield', xtype: 'displayfield',
fieldLabel: gettext('Saved User name'), fieldLabel: gettext('Saved User Name:'),
labelAlign: 'left', labelWidth: 'auto',
labelWidth: '50%',
stateId: 'login-username', stateId: 'login-username',
reference: 'savedUserName', reference: 'savedUserName',
flex: 1,
value: '' value: ''
}, },
{ {
xtype: 'button', xtype: 'button',
cls: 'x-btn-default-toolbar-small proxmox-inline-button', cls: 'x-btn-default-toolbar-small proxmox-inline-button',
text: gettext('Clear User name'), text: gettext('Reset'),
width: 'auto', name: 'clear-username',
name: 'clear-username' },
]
}, },
{ {
xtype: 'box', xtype: 'box',
autoEl: { tag: 'hr'} autoEl: { tag: 'hr'}
}, },
{ {
xtype: 'panel', xtype: 'container',
border: false, layout: 'hbox',
layout: {
type: 'hbox',
align: 'stretch'
},
items: [ items: [
{ {
xtype: 'displayfield', xtype: 'displayfield',
fieldLabel: gettext('Layout'), fieldLabel: gettext('Layout:'),
flex: 2, flex: 1,
}, },
{ {
xtype: 'button', xtype: 'button',
cls: 'x-btn-default-toolbar-small proxmox-inline-button', cls: 'x-btn-default-toolbar-small proxmox-inline-button',
text: gettext('Reset Layout'), text: gettext('Reset'),
tooltip: gettext('Reset all layout changes (for example, column widths)'), tooltip: gettext('Reset all layout changes (for example, column widths)'),
name: 'reset', name: 'reset',
flex: 1,
}, },
] ]
}, },