[pve-devel] applied: [PATCH manager] fix #4370: ui: fix regression of preselecting tree element on login

Thomas Lamprecht t.lamprecht at proxmox.com
Mon Nov 28 18:01:40 CET 2022


Am 28/11/2022 um 13:34 schrieb Dominik Csapak:
> We want to select a tree element based on the fragment in the url after
> a login. For which we must wait for the tree (ResourceStore) to load.
> 
> Since we recently introduced the loading of '/cluster/options' to
> load the overrides of tags, etc. we fire a 'load' event with the
> intention of triggering a rerender of the tree (to include the new
> colors). If that triggers before the ResourceStore is loaded,
> we try to set the element once (which fails) and nothing is selected.
> 
> To fix this, we only trigger it if it was already loaded and is not
> currently loading. Otherwise it will be either redrawn after the ongoing
> load, or by the first load.
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>  www/manager6/Utils.js | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
>

applied, thanks!





More information about the pve-devel mailing list