* [PATCH manager] ui: vnet edit: disable tag and vlan aware fields instead of hiding them
@ 2026-05-19 12:37 Shannon Sterz
0 siblings, 0 replies; only message in thread
From: Shannon Sterz @ 2026-05-19 12:37 UTC (permalink / raw)
To: pve-devel
this improves ux as users unfamiliar with the ui can now tell that
this would be the correct place to set these values, even if the
currently select zone does not support them. particularly important
when creating new vnets, where the controller type is not yet selected
and both fields would be hidden at first.
Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>
---
www/manager6/sdn/VnetEdit.js | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/www/manager6/sdn/VnetEdit.js b/www/manager6/sdn/VnetEdit.js
index 34e382c72..f3d66ce9c 100644
--- a/www/manager6/sdn/VnetEdit.js
+++ b/www/manager6/sdn/VnetEdit.js
@@ -101,18 +101,18 @@ Ext.define('PVE.sdn.VnetInputPanel', {
let tagField = me.down('#sdnVnetTagField');
if (!zoneType || zoneType === 'simple') {
- tagField.setVisible(false);
+ tagField.setDisabled(true);
tagField.setValue('');
} else {
- tagField.setVisible(true);
+ tagField.setDisabled(false);
}
let vlanField = me.down('#sdnVnetVlanAwareField');
if (!zoneType || zoneType === 'evpn') {
- vlanField.setVisible(false);
+ vlanField.setDisabled(true);
vlanField.setValue('');
} else {
- vlanField.setVisible(true);
+ vlanField.setDisabled(false);
}
},
});
--
2.47.3
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2026-05-19 12:38 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-19 12:37 [PATCH manager] ui: vnet edit: disable tag and vlan aware fields instead of hiding them Shannon Sterz
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox