From 423e3cbd185be699a325d60f1e63fe86e51cfe87 Mon Sep 17 00:00:00 2001 From: Dominik Csapak Date: Mon, 22 Feb 2021 14:25:34 +0100 Subject: [PATCH] ui: tape/ChangerStatus: only show loading on 'force refresh' so that we can update in the background without interfering with the users workflow Signed-off-by: Dominik Csapak --- www/tape/ChangerStatus.js | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/www/tape/ChangerStatus.js b/www/tape/ChangerStatus.js index bedeaaec..8e5ee900 100644 --- a/www/tape/ChangerStatus.js +++ b/www/tape/ChangerStatus.js @@ -413,8 +413,10 @@ Ext.define('PBS.TapeManagement.ChangerStatus', { } try { - Proxmox.Utils.setErrorMask(view, true); - Proxmox.Utils.setErrorMask(me.lookup('content')); + if (!use_cache) { + Proxmox.Utils.setErrorMask(view, true); + Proxmox.Utils.setErrorMask(me.lookup('content')); + } let status_fut = PBS.Async.api2({ timeout: 5*60*1000, method: 'GET', @@ -484,10 +486,15 @@ Ext.define('PBS.TapeManagement.ChangerStatus', { me.lookup('import_export').getStore().setData(data['import-export']); me.lookup('drives').getStore().setData(data.drive); - Proxmox.Utils.setErrorMask(view); + if (!use_cache) { + Proxmox.Utils.setErrorMask(view); + } + Proxmox.Utils.setErrorMask(me.lookup('content')); } catch (err) { - Proxmox.Utils.setErrorMask(view); - Proxmox.Utils.setErrorMask(me.lookup('content'), err); + if (!use_cache) { + Proxmox.Utils.setErrorMask(view); + } + Proxmox.Utils.setErrorMask(me.lookup('content'), err.toString()); } },