Lukas Wagner <l.wag...@proxmox.com> writes:

>   - Switch order of 'mailto' and 'mailnotification' field
>   - When mode is 'auto', disable 'mailtnotification' field
>   - When mode is 'auto' and 'mailto' is empty, show
>     hint that the notification system will be used
>
> Signed-off-by: Lukas Wagner <l.wag...@proxmox.com>

Tested both commits, they do what they promise.

> +     let notificationSystemHint = Ext.create({
> +         xtype: 'displayfield',
> +         padding: '0 0 0 5',
> +         userCls: 'pmx-hint',
> +         hidden: true,
> +         value: gettext('No email configured, the notification system will 
> be used'),
> +         viewModel,
> +         bind: {
> +             hidden: '{!hintTextVisible}',

I think the `pmx-hint` being displayed as yellow suggests it is a
warning/error rather than a hint. I wonder if there is a better
approach, as this is certainly not a warning.

Tested-by: Maximiliano Sandoval <m.sando...@proxmox.com>
--
Maximiliano


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to