[pve-devel] [PATCH manager] replication/ui: improve wording for job removal

Thomas Lamprecht t.lamprecht at proxmox.com
Tue Jun 27 15:01:26 CEST 2017


Delete the strange sounding Note from the Removal dialog.
This was added to make sure that an user isn't confused that a Job is
still shown after 'removing' it. But it isn't needed as there is
already noted in the jobs status that it will get removed soon after
doing so, so the user sees that his action had the desired effect, no
extra note needed.

Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
---
 www/manager6/grid/Replication.js | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/www/manager6/grid/Replication.js b/www/manager6/grid/Replication.js
index 66dbf81d..6381c63a 100644
--- a/www/manager6/grid/Replication.js
+++ b/www/manager6/grid/Replication.js
@@ -245,9 +245,7 @@ Ext.define('PVE.grid.ReplicaView', {
 	    handler: 'removeJob',
 	    dangerous: true,
 	    confirmMsg: function(rec) {
-		var msg = Ext.String.format(gettext('Are you sure you want to remove replication job {0}'),
-					    "'" + rec.id  + "'" + '<br>' +
-					    gettext('(Note: Removal of replication job has to be replicated)'));
+		var msg = Ext.String.format(gettext("Are you sure you want to remove replication job '{0}'?"), rec.id);
 		return msg;
 	    },
 	    disabled: true
-- 
2.11.0





More information about the pve-devel mailing list