Fixed - i can resend them all until you had a look at them.

Am 25.04.2013 10:22, schrieb Dietmar Maurer:
> Note: This patch breaks lint:
> 
> [form/NodeSelector.js] Lint at line 29 character 2: Unexpected ','.
> },
> 
>> -----Original Message-----
>> From: pve-devel-boun...@pve.proxmox.com [mailto:pve-devel-
>> boun...@pve.proxmox.com] On Behalf Of Stefan Priebe
>> Sent: Mittwoch, 24. April 2013 11:53
>> To: pve-devel@pve.proxmox.com
>> Subject: [pve-devel] [PATCH 1/9] Nodeselector: change default sort to name
>> instead of mem
>>
>>
>> Signed-off-by: Stefan Priebe <s.pri...@profihost.ag>
>> ---
>>  www/manager/form/NodeSelector.js |   10 ++++++----
>>  1 file changed, 6 insertions(+), 4 deletions(-)
>>
>> diff --git a/www/manager/form/NodeSelector.js
>> b/www/manager/form/NodeSelector.js
>> index 0754cc2..e92dbbd 100644
>> --- a/www/manager/form/NodeSelector.js
>> +++ b/www/manager/form/NodeSelector.js
>> @@ -20,13 +20,13 @@ Ext.define('PVE.form.NodeSelector', {
>>          autoDestory: true,
>>          sorters: [
>>              {
>> +                property : 'node',
>> +                direction: 'ASC'
>> +            },
>> +            {
>>                  property : 'mem',
>>                  direction: 'DESC'
>>              },
>> -            {
>> -                property : 'node',
>> -                direction: 'ASC'
>> -            }
>>          ]
>>      });
>>
>> @@ -39,12 +39,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'
>>                  },
>> --
>> 1.7.10.4
>>
>> _______________________________________________
>> pve-devel mailing list
>> pve-devel@pve.proxmox.com
>> http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> 
> 
_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to