From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68]) by lore.proxmox.com (Postfix) with ESMTPS id B2A161FF16B for ; Fri, 7 Nov 2025 09:56:52 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id C9D57AE08; Fri, 7 Nov 2025 09:56:15 +0100 (CET) From: Stefan Hanreich To: pve-devel@lists.proxmox.com Date: Fri, 7 Nov 2025 09:55:28 +0100 Message-ID: <20251107085553.113655-33-s.hanreich@proxmox.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20251107085553.113655-1-s.hanreich@proxmox.com> References: <20251107085553.113655-1-s.hanreich@proxmox.com> MIME-Version: 1.0 X-SPAM-LEVEL: Spam detection results: 0 AWL -0.182 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment KAM_LAZY_DOMAIN_SECURITY 1 Sending domain does not have any anti-forgery methods RDNS_NONE 0.793 Delivered to internal network by a host with no rDNS SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_NONE 0.001 SPF: sender does not publish an SPF Record Subject: [pve-devel] [PATCH pve-manager v2 2/8] refactor: ui: sdn browser: parametrize zone content panel X-BeenThere: pve-devel@lists.proxmox.com X-Mailman-Version: 2.1.29 Precedence: list List-Id: Proxmox VE development discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: Proxmox VE development discussion Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: pve-devel-bounces@lists.proxmox.com Sender: "pve-devel" In order to reuse the zone content panel for future, additional, panels in the SDN browser, move all static configuration properties into the declaration of the ExtJS widget. In particular, make the on_(de)select callbacks configurable, so this panel can be customized to work with a multitude of additional panels. This is a preparation for extended status reporting for SDN zones, which will add several new panels that want to utilize the content view. The selectionModel uses the default configuration that is included in the GridPanel, so it can be removed. No functional changes intended. Signed-off-by: Stefan Hanreich --- www/manager6/sdn/ZoneContentPanel.js | 11 +++- www/manager6/sdn/ZoneContentView.js | 75 +++++++++++++++------------- 2 files changed, 49 insertions(+), 37 deletions(-) diff --git a/www/manager6/sdn/ZoneContentPanel.js b/www/manager6/sdn/ZoneContentPanel.js index 76b8500b1..50d2dd741 100644 --- a/www/manager6/sdn/ZoneContentPanel.js +++ b/www/manager6/sdn/ZoneContentPanel.js @@ -18,12 +18,21 @@ Ext.define('PVE.sdn.ZoneContentPanel', { var vnetview_panel = Ext.createWidget('pveSDNZoneContentView', { title: 'VNets', region: 'west', - permissions_panel: permissions_panel, + sub_panel: permissions_panel, nodename: me.nodename, zone: me.zone, width: '50%', border: false, split: true, + + on_select: function (_sm, rec) { + let path = `/sdn/zones/${me.zone}/${rec.data.vnet}`; + permissions_panel.setPath(path); + }, + + on_deselect: function () { + permissions_panel.setPath(undefined); + }, }); Ext.apply(me, { diff --git a/www/manager6/sdn/ZoneContentView.js b/www/manager6/sdn/ZoneContentView.js index 97b3612e1..2243f277f 100644 --- a/www/manager6/sdn/ZoneContentView.js +++ b/www/manager6/sdn/ZoneContentView.js @@ -16,6 +16,43 @@ Ext.define( groupHeaderTpl: '{name} ({rows.length} Item{[values.rows.length > 1 ? "s" : ""]})', }, ], + + sub_panel: null, + + columns: [ + { + header: gettext('VNet'), + width: 100, + sortable: true, + dataIndex: 'vnet', + }, + { + header: gettext('Alias'), + width: 300, + sortable: true, + dataIndex: 'alias', + }, + { + header: gettext('Status'), + width: 100, + sortable: true, + dataIndex: 'status', + }, + { + header: gettext('Details'), + flex: 1, + dataIndex: 'statusmsg', + }, + ], + + on_select: function (selectionModel, record) { + // do nothing by default + }, + + on_deselect: function () { + // do nothing by default + }, + initComponent: function () { var me = this; @@ -44,8 +81,6 @@ Ext.define( }, }); - var sm = Ext.create('Ext.selection.RowModel', {}); - var reload = function () { store.load(); }; @@ -53,43 +88,11 @@ Ext.define( Proxmox.Utils.monStoreErrors(me, store); Ext.apply(me, { store: store, - selModel: sm, - tbar: [], - columns: [ - { - header: 'VNet', - width: 100, - sortable: true, - dataIndex: 'vnet', - }, - { - header: 'Alias', - width: 300, - sortable: true, - dataIndex: 'alias', - }, - { - header: gettext('Status'), - width: 100, - sortable: true, - dataIndex: 'status', - }, - { - header: gettext('Details'), - flex: 1, - dataIndex: 'statusmsg', - }, - ], listeners: { activate: reload, show: reload, - select: function (_sm, rec) { - let path = `/sdn/zones/${me.zone}/${rec.data.vnet}`; - me.permissions_panel.setPath(path); - }, - deselect: function () { - me.permissions_panel.setPath(undefined); - }, + select: me.on_select, + deselect: me.on_deselect, }, }); store.load(); -- 2.47.3 _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel