Re: [pve-devel] [PATCH widget-toolkit v4] Fix #582: Add delay to button

2019-07-24 Thread Thomas Lamprecht
On 7/24/19 12:14 PM, Dominik Csapak wrote: >> @@ -130,9 +133,11 @@ Ext.define('Proxmox.button.StdRemoveButton', { >>     handler: function(btn, event, rec) { >>   var me = this; >> +var url = me.getUrl(rec); >> +if (typeof me.delay !== 'undefined') url += "?delay=" + me.delay; > >

Re: [pve-devel] [PATCH widget-toolkit v4] Fix #582: Add delay to button

2019-07-24 Thread Dominik Csapak
looks okay, one comment inline On 7/15/19 12:28 PM, Dominic Jäger wrote: The StdRemoveButton can now pass a delay parameter to the API. It is set undefined as default so that users of the button can set the parameter themselves. Signed-off-by: Dominic Jäger --- v3->v4: Make the delay undefined

[pve-devel] [PATCH widget-toolkit v4] Fix #582: Add delay to button

2019-07-15 Thread Dominic Jäger
The StdRemoveButton can now pass a delay parameter to the API. It is set undefined as default so that users of the button can set the parameter themselves. Signed-off-by: Dominic Jäger --- v3->v4: Make the delay undefined/optional button/Button.js | 7 ++- 1 file changed, 6 insertions(+), 1