[pve-devel] [PATCH-SERIES manager 0/2] ui: qemu: create wizard: improve OS tab

[pve-devel] [PATCH-SERIES manager 0/2] ui: qemu: create wizard: improve OS tab

[Fiona Ebner]

New users are often confused by the storage selection in the 'OS' tab
of the create wizard, the latest case being [0]. Add a heading clearly
and explicitly mentioning what the selection is for. Additionally,
make it fully explicit that storage is for the ISO.

Fiona Ebner (2):
  ui: iso selector: explicitly note that the storage is for the ISO
  ui: qemu: create wizard: provide context for OS ISO selection

 www/manager6/form/IsoSelector.js  |  2 +-
 www/manager6/qemu/CreateWizard.js | 21 +++++++++++++++------
 2 files changed, 16 insertions(+), 7 deletions(-)

-- 
2.47.3



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH manager 1/2] ui: iso selector: explicitly note that the storage is for the ISO

[Fiona Ebner]

New users are often confused by the storage selection in the 'OS' tab
of the create wizard, the latest case being [0]. Making it fully
explicit that storage is for the ISO should help to avoid this
confusion.

[0]: https://forum.proxmox.com/threads/179126/post-831215

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
---
 www/manager6/form/IsoSelector.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/www/manager6/form/IsoSelector.js b/www/manager6/form/IsoSelector.js
index b2d94ed3..a8a18cce 100644
--- a/www/manager6/form/IsoSelector.js
+++ b/www/manager6/form/IsoSelector.js
@@ -70,7 +70,7 @@ Ext.define('PVE.form.IsoSelector', {
             xtype: 'pveStorageSelector',
             reference: 'storage',
             isFormField: false,
-            fieldLabel: gettext('Storage'),
+            fieldLabel: gettext('ISO Storage'),
             storageContent: 'iso',
             allowBlank: false,
             cbind: {
-- 
2.47.3



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

[pve-devel] [PATCH manager 2/2] ui: qemu: create wizard: provide context for OS ISO selection

[Fiona Ebner]

New users are often confused by the storage selection in the 'OS' tab
of the create wizard, the latest case being [0]. Add a heading clearly
and explicitly mentioning what the selection is for.

[0]: https://forum.proxmox.com/threads/179126/post-831215

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
---
 www/manager6/qemu/CreateWizard.js | 21 +++++++++++++++------
 1 file changed, 15 insertions(+), 6 deletions(-)

diff --git a/www/manager6/qemu/CreateWizard.js b/www/manager6/qemu/CreateWizard.js
index 341324c8..451916ee 100644
--- a/www/manager6/qemu/CreateWizard.js
+++ b/www/manager6/qemu/CreateWizard.js
@@ -192,12 +192,21 @@ Ext.define('PVE.qemu.CreateWizard', {
             title: gettext('OS'),
             items: [
                 {
-                    xtype: 'pveQemuCDInputPanel',
-                    bind: {
-                        nodename: '{nodename}',
-                    },
-                    confid: 'ide2',
-                    insideWizard: true,
+                    xtype: 'container',
+                    items: [
+                        {
+                            xtype: 'displayfield',
+                            value: gettext('Select operating system (installer) CD/DVD') + ':',
+                        },
+                        {
+                            xtype: 'pveQemuCDInputPanel',
+                            bind: {
+                                nodename: '{nodename}',
+                            },
+                            confid: 'ide2',
+                            insideWizard: true,
+                        },
+                    ],
                 },
                 {
                     xtype: 'pveQemuOSTypePanel',
-- 
2.47.3



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel