[pve-devel] [PATCH] Nodeselector: change default sort to name instead of mem

Stefan Priebe - Profihost AG s.priebe at profihost.ag
Wed Apr 24 08:17:04 CEST 2013


Hi,

sorry also based on PVE 2.3 instead of current master. Another question
before rebasing.

Do you see a problem in using the current master with squeeze? (sure i
need to revert some binary path changes). I'll stay on Squeeze until summer.

Stefan

Am 24.04.2013 06:02, schrieb Dietmar Maurer:
> I also get errors when I try to apply that patch.
> 
> error: patch failed: www/manager/form/NodeSelector.js:79
> error: www/manager/form/NodeSelector.js: patch does not apply
> 
>> -----Original Message-----
>> From: pve-devel-bounces at pve.proxmox.com [mailto:pve-devel-
>> bounces at pve.proxmox.com] On Behalf Of Stefan Priebe
>> Sent: Montag, 22. April 2013 10:34
>> To: pve-devel at pve.proxmox.com
>> Subject: [pve-devel] [PATCH] Nodeselector: change default sort to name instead
>> of mem
>>
>>
>> Signed-off-by: Stefan Priebe <s.priebe at profihost.ag>
>> ---
>>  www/manager/form/NodeSelector.js |   12 +++++++-----
>>  1 file changed, 7 insertions(+), 5 deletions(-)
>>
>> diff --git a/www/manager/form/NodeSelector.js
>> b/www/manager/form/NodeSelector.js
>> index 5dc06c3..a472d7f 100644
>> --- a/www/manager/form/NodeSelector.js
>> +++ b/www/manager/form/NodeSelector.js
>> @@ -18,13 +18,13 @@ Ext.define('PVE.form.NodeSelector', {
>>  	    autoDestory: true,
>>  	    sorters: [
>>  		{
>> +		    property : 'node',
>> +		    direction: 'ASC'
>> +		},
>> +		{
>>  		    property : 'mem',
>>  		    direction: 'DESC'
>>  		},
>> -		{
>> -		    property : 'node',
>> -		    direction: 'ASC'
>> -		}
>>  	    ]
>>  	});
>>
>> @@ -37,12 +37,14 @@ Ext.define('PVE.form.NodeSelector', {
>>  		    {
>>  			header: 'Node',
>>  			dataIndex: 'node',
>> +			sortable: true,
>>  			hideable: false,
>>  			flex: 1
>>  		    },
>>  		    {
>>  			header: 'Memory usage',
>>  			renderer: PVE.Utils.render_mem_usage,
>> +			sortable: true,
>>  			width: 100,
>>  			dataIndex: 'mem'
>>  		    },
>> @@ -79,4 +81,4 @@ Ext.define('PVE.form.NodeSelector', {
>>
>>          me.callParent();
>>      }
>> -});
>> \ No newline at end of file
>> +});
>> --
>> 1.7.10.4
>>
>> _______________________________________________
>> pve-devel mailing list
>> pve-devel at pve.proxmox.com
>> http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> 
> 



More information about the pve-devel mailing list