[pve-devel] [PATCH pve-manager 3/3] ext6migrate: trigger working store reload when the component is rendered

Emmanuel Kasper e.kasper at proxmox.com
Mon Nov 23 12:43:25 CET 2015


> > This fixes the following problem:
> > After a manual browser refresh, widgets were displayed without data
> 
> >  	    listeners: {
> > -		show: reload
> > +		render: reload
> >  	    }
> >  	});
> 

So I kept digging on this issue and it seems the proper event to monitor would
be 'activate' which is fired when the tabpanel is navigated, or when the
tabpanel is loaded after a browser refresh.

    listeners: {
	activate: reload,
    }

If we do 
this.callParent();
reload();
this works too
but this has the side effect of loading all the content of the tabs, before they
are displayed

> We have a few places where we use:
> 
> fireEvent('show', ...)
> 
> Does that still work with this changes?

We have fireEvent('show' in ConfigPanel which can be leaved as it is, as here we
don't have the refresh bug.
I haven't found other cases where the target of a fireEvent would be a tab
inside a tabpanel.

I will prepare a new version of the patch with all of this.




More information about the pve-devel mailing list