[pve-devel] [PATCH manager] ui: FirewallRules.js: fix #3049 wider rule number column

Aaron Lauterer a.lauterer at proxmox.com
Thu Oct 1 17:22:47 CEST 2020



On 10/1/20 5:10 PM, Thomas Lamprecht wrote:
> On 01.10.20 14:18, Aaron Lauterer wrote:
>> Allows 2 digits rule numbers to be visible in the column. Made it a bit
>> wider than needed to account for potentially wider font rendering on
>> different platforms.
>>
>> Signed-off-by: Aaron Lauterer <a.lauterer at proxmox.com>
>> ---
>>   www/manager6/grid/FirewallRules.js | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/www/manager6/grid/FirewallRules.js b/www/manager6/grid/FirewallRules.js
>> index ec2d1c84..42035129 100644
>> --- a/www/manager6/grid/FirewallRules.js
>> +++ b/www/manager6/grid/FirewallRules.js
>> @@ -629,7 +629,7 @@ Ext.define('PVE.FirewallRules', {
>>   		// similar to xtype: 'rownumberer',
>>   		dataIndex: 'pos',
>>   		resizable: false,
>> -		width: 23,
>> +		width: 42,
>>   		sortable: false,
>>   		align: 'right',
>>   		hideable: false,
>>
> 
> oh, and maybe you can try to switch that whole column definition away from fixed
> widths to relative "flex" only. Just needs to look OK on 720p and quite good on
> 1080p. It's not always possible, but most of the time it works out well and not
> using fixed widths is always nicer, IMO.
> 

good point. I'll take a look at it.





More information about the pve-devel mailing list