[pve-devel] [PATCH manager] Bugfix Only set a content when we have a selection

Emmanuel Kasper e.kasper at proxmox.com
Wed Sep 14 09:17:44 CEST 2016


Removing a node via updateTree() would cause the selection to be lost temporary
which in turn via the selectionchange listener would select an
empty content.
Now we only set a content if we have a selection.
This fix a graphical glitch where a migrated VM would not be properly
re-selected at its new position in tree.

The bug was introduced by:
    Clear sensitive content when logging out fixes #1060
    192f1ad2827c895bfd91465dfa4d3537ce02e9d2
---
 www/manager6/Workspace.js | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/www/manager6/Workspace.js b/www/manager6/Workspace.js
index 8b7b239..4c57267 100644
--- a/www/manager6/Workspace.js
+++ b/www/manager6/Workspace.js
@@ -313,9 +313,8 @@ Ext.define('PVE.StdWorkspace', {
 				viewFilter: selview.getViewFilter()
 			    };
 			    PVE.curSelectedNode = n;
+			    me.setContent(comp);
 			}
-
-			me.setContent(comp);
 		    }
 		}
 	    }
-- 
2.1.4





More information about the pve-devel mailing list