[pve-devel] [PATCH manager 4/4] wizard: merge CD/DVD and OS panels
Thomas Lamprecht
t.lamprecht at proxmox.com
Tue Aug 22 11:57:42 CEST 2017
We had a single column for the CD/DVD panel, fill the second column
with the now in size reduced OS type selector, as ISO selection and
OS type selection are tied together.
Reduces steps needed for creating a VM.
Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
---
www/manager6/qemu/CDEdit.js | 6 +-----
www/manager6/qemu/CreateWizard.js | 34 ++++++++++++++++++++++------------
2 files changed, 23 insertions(+), 17 deletions(-)
diff --git a/www/manager6/qemu/CDEdit.js b/www/manager6/qemu/CDEdit.js
index 51921b8d..aa359dc6 100644
--- a/www/manager6/qemu/CDEdit.js
+++ b/www/manager6/qemu/CDEdit.js
@@ -135,11 +135,7 @@ Ext.define('PVE.qemu.CDInputPanel', {
boxLabel: gettext('Do not use any media')
});
- if (me.insideWizard) {
- me.column1 = items;
- } else {
- me.items = items;
- }
+ me.items = items;
me.callParent();
}
diff --git a/www/manager6/qemu/CreateWizard.js b/www/manager6/qemu/CreateWizard.js
index 126925e2..c9ed1bdd 100644
--- a/www/manager6/qemu/CreateWizard.js
+++ b/www/manager6/qemu/CreateWizard.js
@@ -86,19 +86,29 @@ Ext.define('PVE.qemu.CreateWizard', {
}
},
{
- xtype: 'pveQemuCDInputPanel',
- reference: 'cdpanel',
- title: gettext('CD/DVD'),
- confid: 'ide2',
- fieldDefaults: {
- labelWidth: 160
+ xtype: 'container',
+ layout: {
+ type: 'hbox',
+ align: 'stretch'
},
- insideWizard: true
- },
- {
- title: gettext('OS'),
- xtype: 'pveQemuOSTypePanel',
- insideWizard: true
+ defaults: {
+ padding: '0 10',
+ },
+ title: gettext('OS Selection'),
+ items: [
+ {
+ xtype: 'pveQemuCDInputPanel',
+ reference: 'cdpanel',
+ confid: 'ide2',
+ flex: 1,
+ insideWizard: true
+ },
+ {
+ xtype: 'pveQemuOSTypePanel',
+ flex: 1,
+ insideWizard: true
+ }
+ ]
},
{
xtype: 'pveQemuHDInputPanel',
--
2.11.0
More information about the pve-devel
mailing list