[pve-devel] [PATCH pve-manager 3/3] Fix LXC summary view:

Emmanuel Kasper e.kasper at proxmox.com
Mon Mar 14 13:35:11 CET 2016


 * replace scrollable with autoScroll and move to prototype body
 * use 'activate' to load store on F5
 * do not set a height on the StatusView component: it hides some rows,
and the framework sets a good working default height
---
 www/manager6/lxc/StatusView.js |  1 -
 www/manager6/lxc/Summary.js    | 15 ++++++++-------
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/www/manager6/lxc/StatusView.js b/www/manager6/lxc/StatusView.js
index 20da0ec..d71420c 100644
--- a/www/manager6/lxc/StatusView.js
+++ b/www/manager6/lxc/StatusView.js
@@ -70,7 +70,6 @@ Ext.define('PVE.lxc.StatusView', {
 
 	Ext.applyIf(me, {
 	    cwidth1: 150,
-	    height: 200,
 	    rows: rows
 	});
 
diff --git a/www/manager6/lxc/Summary.js b/www/manager6/lxc/Summary.js
index 9ad19b5..878f546 100644
--- a/www/manager6/lxc/Summary.js
+++ b/www/manager6/lxc/Summary.js
@@ -2,6 +2,13 @@ Ext.define('PVE.lxc.Summary', {
     extend: 'Ext.panel.Panel',
     alias: 'widget.pveLxcSummary',
 
+    scrollable: true,
+    bodyStyle: 'padding:10px',
+    defaults: {
+	style: 'padding-top:10px',
+	width: 800
+    },
+
     initComponent: function() {
         var me = this;
 
@@ -46,12 +53,6 @@ Ext.define('PVE.lxc.Summary', {
 		    xtype: 'pveRRDTypeSelector'
 		}
 	    ],
-	    autoScroll: true,
-	    bodyStyle: 'padding:10px',
-	    defaults: {
-		style: 'padding-top:10px',
-		width: 800
-	    },		
 	    items: [
 		{
 		    style: 'padding-top:0px',
@@ -93,7 +94,7 @@ Ext.define('PVE.lxc.Summary', {
 	    ]
 	});
 
-	me.on('show', function() {
+	me.on('activate', function() {
 	    notesview.load();
 	});
 
-- 
2.1.4





More information about the pve-devel mailing list