public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Dominik Csapak <d.csapak@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: [PATCH manager v3 13/13] ui: qemu: hardware view: inline 'add efi' menuitem
Date: Fri, 15 May 2026 10:44:31 +0200	[thread overview]
Message-ID: <20260515085349.1123127-14-d.csapak@proxmox.com> (raw)
In-Reply-To: <20260515085349.1123127-1-d.csapak@proxmox.com>

so it's in line with the remaining add buttons.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 www/manager6/qemu/HardwareView.js | 43 ++++++++++++++++---------------
 1 file changed, 22 insertions(+), 21 deletions(-)

diff --git a/www/manager6/qemu/HardwareView.js b/www/manager6/qemu/HardwareView.js
index e16436ab..21914a29 100644
--- a/www/manager6/qemu/HardwareView.js
+++ b/www/manager6/qemu/HardwareView.js
@@ -502,25 +502,6 @@ Ext.define('PVE.qemu.HardwareView', {
                 'Otherwise, you will be prompted for the BitLocker recovery key on the next boot!',
             );
 
-        let efidisk_menuitem = Ext.create('Ext.menu.Item', {
-            text: gettext('EFI Disk'),
-            iconCls: 'fa fa-fw fa-hdd-o black',
-            disabled: !caps.vms['VM.Config.Disk'],
-            handler: function () {
-                let { data: bios } = me.rstore.getData().map.bios || {};
-
-                Ext.create('PVE.qemu.EFIDiskEdit', {
-                    autoShow: true,
-                    url: '/api2/extjs/' + baseurl,
-                    pveSelNode: me.pveSelNode,
-                    usesEFI: bios?.value === 'ovmf' || bios?.pending === 'ovmf',
-                    listeners: {
-                        destroy: () => me.reload(),
-                    },
-                });
-            },
-        });
-
         let counts = {};
         let isAtLimit = (type) => counts[type] >= PVE.Utils.hardware_counts[type];
         let isAtUsbLimit = () => {
@@ -560,7 +541,7 @@ Ext.define('PVE.qemu.HardwareView', {
             me.down('#addSerial').setDisabled(noVMConfigHWTypePerm || isAtLimit('serial'));
             me.down('#addNet').setDisabled(noVMConfigNetPerm || isAtLimit('net'));
             me.down('#addRng').setDisabled(noVMConfigHWTypePerm || isAtLimit('rng'));
-            efidisk_menuitem.setDisabled(noVMConfigDiskPerm || isAtLimit('efidisk'));
+            me.down('#addEfi').setDisabled(noVMConfigDiskPerm || isAtLimit('efidisk'));
             me.down('#addTpmState').setDisabled(noVMConfigDiskPerm || isAtLimit('tpmstate'));
             me.down('#addVirtiofs').setDisabled(noVMConfigOptionsPerm || isAtLimit('virtiofs'));
             me.down('#addCloudinitDrive').setDisabled(
@@ -842,7 +823,27 @@ Ext.define('PVE.qemu.HardwareView', {
                                         disabled: !caps.vms['VM.Config.Disk'],
                                         handler: editorFactory('HDEdit', { importDisk: true }),
                                     },
-                                    efidisk_menuitem,
+                                    {
+                                        itemId: 'addEfi',
+                                        text: gettext('EFI Disk'),
+                                        iconCls: 'fa fa-fw fa-hdd-o black',
+                                        disabled: !caps.vms['VM.Config.Disk'],
+                                        handler: function () {
+                                            let { data: bios } = me.rstore.getData().map.bios || {};
+
+                                            Ext.create('PVE.qemu.EFIDiskEdit', {
+                                                autoShow: true,
+                                                url: '/api2/extjs/' + baseurl,
+                                                pveSelNode: me.pveSelNode,
+                                                usesEFI:
+                                                    bios?.value === 'ovmf' ||
+                                                    bios?.pending === 'ovmf',
+                                                listeners: {
+                                                    destroy: () => me.reload(),
+                                                },
+                                            });
+                                        },
+                                    },
                                     {
                                         text: gettext('TPM State'),
                                         itemId: 'addTpmState',
-- 
2.47.3





  parent reply	other threads:[~2026-05-15  8:54 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-05-15  8:44 [PATCH manager v3 00/13] ui: split out disks and nics into grids Dominik Csapak
2026-05-15  8:44 ` [PATCH manager v3 01/13] ui: utils: factor out 'media=cdrom' check Dominik Csapak
2026-05-15  8:44 ` [PATCH manager v3 02/13] ui: factor out the guest key nic regex check Dominik Csapak
2026-05-15  8:44 ` [PATCH manager v3 03/13] ui: parser: qemu drive: allow '-' in key names Dominik Csapak
2026-05-15  8:44 ` [PATCH manager v3 04/13] ui: add pending grid Dominik Csapak
2026-05-15  8:44 ` [PATCH manager v3 05/13] ui: revert button: add parentXType and reloadCallback Dominik Csapak
2026-05-15  8:44 ` [PATCH manager v3 06/13] ui: button: add config remove button Dominik Csapak
2026-05-15  8:44 ` [PATCH manager v3 07/13] ui: qemu: hardware: wrap in container Dominik Csapak
2026-05-15  8:44 ` [PATCH manager v3 08/13] ui: qemu: introduce hardware disk grid Dominik Csapak
2026-05-15  8:44 ` [PATCH manager v3 09/13] ui: qemu: introduce hardware net grid Dominik Csapak
2026-05-15  8:44 ` [PATCH manager v3 10/13] ui: qemu: hardware view: separate disks into own grid Dominik Csapak
2026-05-15  8:44 ` [PATCH manager v3 11/13] ui: qemu: hardware view: separate nics " Dominik Csapak
2026-05-15  8:44 ` [PATCH manager v3 12/13] ui: qemu: hardware view: inline edit/remove/revert button in general grid Dominik Csapak
2026-05-15  8:44 ` Dominik Csapak [this message]
2026-05-15  9:18 ` [PATCH manager v3 00/13] ui: split out disks and nics into grids Dominik Csapak
2026-05-15  9:20   ` Dominik Csapak

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260515085349.1123127-14-d.csapak@proxmox.com \
    --to=d.csapak@proxmox.com \
    --cc=pve-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal