all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal