[pbs-devel] applied: [PATCH proxmox-backup 1/2] ui: datastore/Content: reload in activate listener
Dietmar Maurer
dietmar at proxmox.com
Mon Oct 4 10:29:50 CEST 2021
applied both patches
On 10/1/21 11:56 AM, Dominik Csapak wrote:
> when we trigger the first load before the panel was fully created,
> there was no load mask for it (but the snapshots would "pop in" on load)
>
> move the first reload into the 'activate' listener. this will be called
> the every time a user opens the content tab of a datastore, so guard
> it by a 'firstLoad' bool.
>
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
> www/datastore/Content.js | 12 +++++++++++-
> 1 file changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/www/datastore/Content.js b/www/datastore/Content.js
> index 57693785..ddc923fd 100644
> --- a/www/datastore/Content.js
> +++ b/www/datastore/Content.js
> @@ -73,7 +73,6 @@ Ext.define('PBS.DataStoreContent', {
> 'backup-time',
> ]);
> Proxmox.Utils.monStoreErrors(view, this.store);
> - this.reload(); // initial load
> },
>
> reload: function() {
> @@ -625,6 +624,17 @@ Ext.define('PBS.DataStoreContent', {
> },
> },
>
> + listeners: {
> + activate: function() {
> + let me = this;
> + // only load on first activate to not load every tab switch
> + if (!me.firstLoad) {
> + me.getController().reload();
> + me.firstLoad = true;
> + }
> + },
> + },
> +
> viewConfig: {
> getRowClass: function(record, index) {
> let verify = record.get('verification');
More information about the pbs-devel
mailing list