From 120b530a31f44060cbb931e37834f5992be2daac Mon Sep 17 00:00:00 2001 From: Thomas Lamprecht Date: Wed, 10 Apr 2024 12:18:56 +0200 Subject: [PATCH] notify: shorten text to "reset" for built-in targets While the difference between Reset and Remove is a bit more subtle this also leads to less jumping around of UI elements on the right to it (we normally avoid such size-changes that cause layout changes completely). Also, the confirmation message is quite telling, so this is not too bad. Signed-off-by: Thomas Lamprecht --- src/panel/NotificationConfigView.js | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/src/panel/NotificationConfigView.js b/src/panel/NotificationConfigView.js index 2e2eede..9505eb7 100644 --- a/src/panel/NotificationConfigView.js +++ b/src/panel/NotificationConfigView.js @@ -8,13 +8,7 @@ Ext.define('Proxmox.panel.NotificationConfigViewModel', { let origin = get('selection')?.get('origin'); return origin === 'modified-builtin' || origin === 'builtin'; }, - removeButtonText: function(get) { - if (get('builtinSelected')) { - return gettext('Reset to default'); - } else { - return gettext('Remove'); - } - }, + removeButtonText: get => get('builtinSelected') ? gettext('Reset') : gettext('Remove'), removeButtonConfirmMessage: function(get) { if (get('builtinSelected')) { return gettext('Do you want to reset {0} to its default settings?');