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 <t.lamprecht@proxmox.com>
This commit is contained in:
Thomas Lamprecht 2024-04-10 12:18:56 +02:00
parent 83bae06cae
commit 120b530a31

View File

@ -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?');