[pve-devel] [PATCH manager] ui: add comment about disabling spice/xterm initially

[Dominik Csapak]

Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
@thomas: tried again to reproduce your issue, but could not, choosing
a specific console type in datacenter->options resulted in the ui
choosing that (when available) or falling back to the 'best' method
available

did you maybe try from a different window without refreshing?
we only check that value once in the beginning, or when you change
it in the same tab

 www/manager6/qemu/Config.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/www/manager6/qemu/Config.js b/www/manager6/qemu/Config.js
index b5f8cc9c..108df69b 100644
--- a/www/manager6/qemu/Config.js
+++ b/www/manager6/qemu/Config.js
@@ -199,6 +199,7 @@ Ext.define('PVE.qemu.Config', {
 	    disabled: !caps.vms['VM.Console'],
 	    hidden: template,
 	    consoleType: 'kvm',
+	    // disable spice/xterm for default action until status api call succeeded
 	    enableSpice: false,
 	    enableXtermjs: false,
 	    consoleName: vm.name,
-- 
2.20.1