We use the field label 'SCSI Controller'. 'Type' is redundant, otherwise we would have Network Card Type, Display Type, Network Card type, etc ... --- changes since v4: * rename the field label to SCSI Controller, and the associated component reference to scsiController.
www/manager6/qemu/HDEdit.js | 15 ++++++++++++--- www/manager6/qemu/Options.js | 2 +- 2 files changed, 13 insertions(+), 4 deletions(-) diff --git a/www/manager6/qemu/HDEdit.js b/www/manager6/qemu/HDEdit.js index 8a415d8b..b6f08080 100644 --- a/www/manager6/qemu/HDEdit.js +++ b/www/manager6/qemu/HDEdit.js @@ -24,11 +24,12 @@ Ext.define('PVE.qemu.HDInputPanel', { this.lookup('iothread').setValue(false); } - var allowDiscard = value.match(/^scsi/); - this.lookup('discard').setDisabled(!allowDiscard); - if (!allowDiscard) { + var scsi = value.match(/^scsi/); + this.lookup('discard').setDisabled(!scsi); + if (!scsi) { this.lookup('discard').setValue(false); } + this.lookup('scsiController').setVisible(scsi); }, control: { @@ -141,6 +142,7 @@ Ext.define('PVE.qemu.HDInputPanel', { if (me.bussel) { me.bussel.setVMConfig(vmconfig); + me.scsiController.setValue(vmconfig.scsihw); } if (me.unusedDisks) { var disklist = []; @@ -195,6 +197,13 @@ Ext.define('PVE.qemu.HDInputPanel', { vmconfig: me.insideWizard ? {ide2: 'cdrom'} : {} }); me.column1.push(me.bussel); + me.scsiController = Ext.create('Ext.form.field.Display', { + fieldLabel: gettext('SCSI Controller'), + reference: 'scsiController', + renderer: PVE.Utils.render_scsihw, + hidden: true + }); + me.column1.push(me.scsiController); } if (me.unused) { diff --git a/www/manager6/qemu/Options.js b/www/manager6/qemu/Options.js index 9dc36d70..b5d32537 100644 --- a/www/manager6/qemu/Options.js +++ b/www/manager6/qemu/Options.js @@ -175,7 +175,7 @@ Ext.define('PVE.qemu.Options', { } : undefined }, scsihw: { - header: gettext('SCSI Controller Type'), + header: gettext('SCSI Controller'), editor: caps.vms['VM.Config.Options'] ? 'PVE.qemu.ScsiHwEdit' : undefined, renderer: PVE.Utils.render_scsihw, defaultValue: '' -- 2.11.0 _______________________________________________ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel