* [pve-devel] [PATCH pve-manager 1/2] fabric: increase width of fabric edit/add window @ 2025-07-17 12:06 Gabriel Goller 2025-07-17 12:06 ` [pve-devel] [PATCH pve-manager 2/2] fabrics: add emptyText attributes to some input fields Gabriel Goller 2025-07-17 21:28 ` [pve-devel] applied-series: [PATCH pve-manager 1/2] fabric: increase width of fabric edit/add window Thomas Lamprecht 0 siblings, 2 replies; 3+ messages in thread From: Gabriel Goller @ 2025-07-17 12:06 UTC (permalink / raw) To: pve-devel; +Cc: Thomas Lamprecht Increase the width of the OSPF and OpenFabric add and create windows. This looks more pleasant and the input fields aren't that crammed anymore. Suggested-by: Thomas Lamprecht <t.lamprecht@proxmox.com> Signed-off-by: Gabriel Goller <g.goller@proxmox.com> --- www/manager6/sdn/fabrics/FabricEdit.js | 2 ++ 1 file changed, 2 insertions(+) diff --git a/www/manager6/sdn/fabrics/FabricEdit.js b/www/manager6/sdn/fabrics/FabricEdit.js index 3248dc4ce12a..5aa001564175 100644 --- a/www/manager6/sdn/fabrics/FabricEdit.js +++ b/www/manager6/sdn/fabrics/FabricEdit.js @@ -2,6 +2,8 @@ Ext.define('PVE.sdn.Fabric.Fabric.Edit', { extend: 'Proxmox.window.Edit', mixins: ['Proxmox.Mixin.CBind'], + width: 400, + fabricId: undefined, baseUrl: '/cluster/sdn/fabrics/fabric', -- 2.39.5 _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel ^ permalink raw reply [flat|nested] 3+ messages in thread
* [pve-devel] [PATCH pve-manager 2/2] fabrics: add emptyText attributes to some input fields 2025-07-17 12:06 [pve-devel] [PATCH pve-manager 1/2] fabric: increase width of fabric edit/add window Gabriel Goller @ 2025-07-17 12:06 ` Gabriel Goller 2025-07-17 21:28 ` [pve-devel] applied-series: [PATCH pve-manager 1/2] fabric: increase width of fabric edit/add window Thomas Lamprecht 1 sibling, 0 replies; 3+ messages in thread From: Gabriel Goller @ 2025-07-17 12:06 UTC (permalink / raw) To: pve-devel; +Cc: Thomas Lamprecht Add default emptyText attributes to OpenFabric and OSPF add/edit windows. Use the default values on Hello Interval [0], CSNP Interval [1] and Hello Multiplier [2] inputs. For the OSPF area just use the simplest area "0" (the backbone). [0]: https://github.com/FRRouting/frr/blob/f56119ea0321ee7d149172422a6d87e106a94778/isisd/isis_constants.h#L77 [1]: https://github.com/FRRouting/frr/blob/f56119ea0321ee7d149172422a6d87e106a94778/isisd/isis_constants.h#L69 [2]: https://github.com/FRRouting/frr/blob/f56119ea0321ee7d149172422a6d87e106a94778/isisd/isis_constants.h#L81 Suggested-by: Thomas Lamprecht <t.lamprecht@proxmox.com> Signed-off-by: Gabriel Goller <g.goller@proxmox.com> --- www/manager6/sdn/fabrics/openfabric/FabricEdit.js | 2 ++ www/manager6/sdn/fabrics/openfabric/InterfacePanel.js | 1 + www/manager6/sdn/fabrics/ospf/FabricEdit.js | 1 + 3 files changed, 4 insertions(+) diff --git a/www/manager6/sdn/fabrics/openfabric/FabricEdit.js b/www/manager6/sdn/fabrics/openfabric/FabricEdit.js index 9da624190bc6..46dd61c410aa 100644 --- a/www/manager6/sdn/fabrics/openfabric/FabricEdit.js +++ b/www/manager6/sdn/fabrics/openfabric/FabricEdit.js @@ -47,6 +47,7 @@ Ext.define('PVE.sdn.Fabric.OpenFabric.Fabric.Edit', { labelWidth: 120, name: 'hello_interval', allowBlank: true, + emptyText: '3', skipEmptyText: true, cbind: { deleteEmpty: '{!isCreate}', @@ -58,6 +59,7 @@ Ext.define('PVE.sdn.Fabric.OpenFabric.Fabric.Edit', { labelWidth: 120, name: 'csnp_interval', allowBlank: true, + emptyText: '10', skipEmptyText: true, cbind: { deleteEmpty: '{!isCreate}', diff --git a/www/manager6/sdn/fabrics/openfabric/InterfacePanel.js b/www/manager6/sdn/fabrics/openfabric/InterfacePanel.js index fbc2a27d8c9e..f23b889b4687 100644 --- a/www/manager6/sdn/fabrics/openfabric/InterfacePanel.js +++ b/www/manager6/sdn/fabrics/openfabric/InterfacePanel.js @@ -24,6 +24,7 @@ Ext.define('PVE.sdn.Fabric.OpenFabric.InterfacePanel', { widget: { xtype: 'proxmoxintegerfield', isFormField: false, + emptyText: '10', bind: { disabled: '{record.isDisabled}', }, diff --git a/www/manager6/sdn/fabrics/ospf/FabricEdit.js b/www/manager6/sdn/fabrics/ospf/FabricEdit.js index c27c6eb4458d..4c4e17c99090 100644 --- a/www/manager6/sdn/fabrics/ospf/FabricEdit.js +++ b/www/manager6/sdn/fabrics/ospf/FabricEdit.js @@ -14,6 +14,7 @@ Ext.define('PVE.sdn.Fabric.Ospf.Fabric.Edit', { fieldLabel: gettext('Area'), labelWidth: 120, name: 'area', + emptyText: '0', allowBlank: false, }, ], -- 2.39.5 _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel ^ permalink raw reply [flat|nested] 3+ messages in thread
* [pve-devel] applied-series: [PATCH pve-manager 1/2] fabric: increase width of fabric edit/add window 2025-07-17 12:06 [pve-devel] [PATCH pve-manager 1/2] fabric: increase width of fabric edit/add window Gabriel Goller 2025-07-17 12:06 ` [pve-devel] [PATCH pve-manager 2/2] fabrics: add emptyText attributes to some input fields Gabriel Goller @ 2025-07-17 21:28 ` Thomas Lamprecht 1 sibling, 0 replies; 3+ messages in thread From: Thomas Lamprecht @ 2025-07-17 21:28 UTC (permalink / raw) To: pve-devel, Gabriel Goller On Thu, 17 Jul 2025 14:06:38 +0200, Gabriel Goller wrote: > Increase the width of the OSPF and OpenFabric add and create windows. > This looks more pleasant and the input fields aren't that crammed > anymore. > > Applied, thanks! [1/2] fabric: increase width of fabric edit/add window commit: dd734f2496a4ca755a73646d8b811bd0b6e8e91b [2/2] fabrics: add emptyText attributes to some input fields commit: 2a13a2e5d9818fe00d779970740a9d98ae2cafcf _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-07-17 21:29 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2025-07-17 12:06 [pve-devel] [PATCH pve-manager 1/2] fabric: increase width of fabric edit/add window Gabriel Goller 2025-07-17 12:06 ` [pve-devel] [PATCH pve-manager 2/2] fabrics: add emptyText attributes to some input fields Gabriel Goller 2025-07-17 21:28 ` [pve-devel] applied-series: [PATCH pve-manager 1/2] fabric: increase width of fabric edit/add window Thomas Lamprecht
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox