[pve-devel] [PATCH manager 4/5] move notepanel padding out of the class

Dominik Csapak d.csapak at proxmox.com
Wed Jun 15 15:13:26 CEST 2016


it makes more sense to define the padding in context,
rather than directly in the component

especially if we want to use this panel elsewhere in the future

Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
also this is necessary for the next patch in this series,
otherwise it looks ugly
 www/manager6/lxc/Summary.js     | 1 +
 www/manager6/panel/NotesView.js | 1 -
 www/manager6/qemu/Summary.js    | 1 +
 3 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/www/manager6/lxc/Summary.js b/www/manager6/lxc/Summary.js
index 6a648cb..8df54e9 100644
--- a/www/manager6/lxc/Summary.js
+++ b/www/manager6/lxc/Summary.js
@@ -38,6 +38,7 @@ Ext.define('PVE.lxc.Summary', {
 
 	var notesview = Ext.create('PVE.panel.NotesView', {
 	    pveSelNode: me.pveSelNode,
+	    padding: '0 0 0 10',
 	    flex: 1
 	});
 
diff --git a/www/manager6/panel/NotesView.js b/www/manager6/panel/NotesView.js
index 0ab174c..029c840 100644
--- a/www/manager6/panel/NotesView.js
+++ b/www/manager6/panel/NotesView.js
@@ -2,7 +2,6 @@ Ext.define('PVE.panel.NotesView', {
     extend: 'Ext.panel.Panel',
 
     title: gettext("Notes"),
-    style: 'padding-left:10px',
     bodyStyle: 'white-space:pre',
     bodyPadding: 10,
     scrollable: true,
diff --git a/www/manager6/qemu/Summary.js b/www/manager6/qemu/Summary.js
index 3b6307e..5de8f79 100644
--- a/www/manager6/qemu/Summary.js
+++ b/www/manager6/qemu/Summary.js
@@ -38,6 +38,7 @@ Ext.define('PVE.qemu.Summary', {
 
 	var notesview = Ext.create('PVE.panel.NotesView', {
 	    pveSelNode: me.pveSelNode,
+	    padding: '0 0 0 10',
 	    flex: 1
 	});
 
-- 
2.1.4





More information about the pve-devel mailing list