[pve-devel] [PATCH widget-toolkit 1/5] APTRepositories: change separation style between panels

Dominik Csapak d.csapak at proxmox.com
Wed Jun 30 17:14:55 CEST 2021


having double borders does not look so nice, separate them with a bit
of padding

also change the warning panel to a simple header

Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
 src/node/APTRepositories.js | 17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)

diff --git a/src/node/APTRepositories.js b/src/node/APTRepositories.js
index 6812d46..7544a23 100644
--- a/src/node/APTRepositories.js
+++ b/src/node/APTRepositories.js
@@ -24,6 +24,8 @@ Ext.define('Proxmox.node.APTRepositoriesErrors', {
 
     store: {},
 
+    border: false,
+
     viewConfig: {
 	stripeRows: false,
 	getRowClass: () => 'proxmox-invalid-row',
@@ -52,6 +54,8 @@ Ext.define('Proxmox.node.APTRepositoriesGrid', {
 
     cls: 'proxmox-apt-repos', // to allow applying styling to general components with local effect
 
+    border: false,
+
     tbar: [
 	{
 	    text: gettext('Reload'),
@@ -350,18 +354,25 @@ Ext.define('Proxmox.node.APTRepositories', {
 
     items: [
 	{
-	    title: gettext('Warning'),
-	    name: 'repositoriesMainWarning',
-	    xtype: 'panel',
+	    xtype: 'header',
+	    baseCls: 'x-panel-header',
 	    bind: {
+		hidden: '{!mainWarning}',
 		title: '{mainWarning}',
+	    },
+	},
+	{
+	    xtype: 'box',
+	    bind: {
 		hidden: '{!mainWarning}',
 	    },
+	    height: 5,
 	},
 	{
 	    xtype: 'proxmoxNodeAPTRepositoriesErrors',
 	    name: 'repositoriesErrors',
 	    hidden: true,
+	    padding: '0 0 5 0',
 	    bind: {
 		hidden: '{noErrors}',
 	    },
-- 
2.30.2






More information about the pve-devel mailing list