most 'Remove' buttons we have used the same code pattern over and over,
with the StdRemoveButton we have a component which does all of this for
us
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
(NB: the error was only displayed intermittently in the browser console)
The reload call could try a store reload before the store has
a defined url.
The call here is useless since the initial display of
the storage selector compoment will call setStorage() to pass an url
to the store and load it.
Also remove the test on me.store.proxy.url since it was not working
(ExtJS autodefined an URL based on the model name)
this patch moves the onlineHelp ids into the javascript classes, instead
of defining them where we use the classes
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
this adds a button to show the configuration of backups
this helps if someone wants to know the details of the
vm in the backup (e.g. name, storages, etc.)
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
currently, as soon as you type something in the backup search bar,
the filter for the vm type (e.g. lxc or qemu) gets cleared, and
you see *all* backups (also those not for the current type)
this patch adds the original filterFn to the filter when searching
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
Using ExtJS6 a browser refresh (F5) do not send the 'show' event which we use to display the store as it did with ExtJS4.
At the momment only for tabs inside the datacenter tabpanel, since node and lxc tabpanels are not yet in manager6/
