ui: sync job: group remote fields and use "Source" in labels

Using "Source" helps to understand that this is a "pull from remote"
sync, not a "push to remote" one.

https://forum.proxmox.com/threads/suggestions-regarding-configurations-terminology.73272/

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

diff --git a/www/window/SyncJobEdit.js b/www/window/SyncJobEdit.js
index 2759716c23b4b64583db476d15c3258048a887df..1c8a6656550ce11aabd0856ff0eb17f1ecda71ff 100644 (file)
--- a/www/window/SyncJobEdit.js
+++ b/www/window/SyncJobEdit.js
@@ -28,7 +28,7 @@ Ext.define('PBS.window.SyncJobEdit', {
        xtype: 'inputpanel',
        column1: [
            {
-               fieldLabel: gettext('Sync Job'),
+               fieldLabel: gettext('Sync Job ID'),
                xtype: 'pmxDisplayEditField',
                name: 'id',
                renderer: Ext.htmlEncode,
@@ -39,22 +39,22 @@ Ext.define('PBS.window.SyncJobEdit', {
                },
            },
            {
-               fieldLabel: gettext('Remote'),
+               fieldLabel: gettext('Source Remote'),
                xtype: 'pbsRemoteSelector',
                allowBlank: false,
                name: 'remote',
            },
            {
-               fieldLabel: gettext('Local Datastore'),
-               xtype: 'pbsDataStoreSelector',
+               fieldLabel: gettext('Source Datastore'),
+               xtype: 'proxmoxtextfield',
                allowBlank: false,
-               name: 'store',
+               name: 'remote-store',
            },
            {
-               fieldLabel: gettext('Remote Datastore'),
-               xtype: 'proxmoxtextfield',
+               fieldLabel: gettext('Local Datastore'),
+               xtype: 'pbsDataStoreSelector',
                allowBlank: false,
-               name: 'remote-store',
+               name: 'store',
            },
        ],
 
@@ -70,6 +70,7 @@ Ext.define('PBS.window.SyncJobEdit', {
                fieldLabel: gettext('Schedule'),
                xtype: 'proxmoxtextfield',
                name: 'schedule',
+               emptyText: gettext('none'),
                cbind: {
                    deleteEmpty: '{!isCreate}',
                },