"window:" by itself doesn't provide much information as a prefix. If you
use "restore window:" then you can drop that from the end of the commit
title.

Am 12.02.25 um 14:02 schrieb Markus Frank:
> @@ -141,9 +144,10 @@ Ext.define('PVE.window.Restore', {
>                           
> view.lookupReference(`${key}Field`).setEmptyText(value);
>                       }
>                   });
> -
> +                 let diskformat = 
> view.down('pveDiskFormatSelector[name=diskformat]');
> +                 diskformat.setVisible(false);

I think it's better to do this in the component itself, see my comment
on the previous patch.

>                   if (!allStoragesAvailable) {
> -                     let storagesel = 
> view.down('pveStorageSelector[name=storage]');
> +                     let storagesel = 
> view.down('pveStorageSelector[name=hdstorage]');
>                       storagesel.allowBlank = false;
>                       storagesel.setEmptyText('');
>                   }

Looks good otherwise :)


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

Reply via email to