ui: improve sync job view layout
Avoid overuse of flex, that is as bad as having all to fixed widths. In spirit similar to the previous commit for the verify panel, see that for some rationale. Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
		@ -224,68 +224,71 @@ Ext.define('PBS.config.SyncJobView', {
 | 
				
			|||||||
 | 
					
 | 
				
			||||||
    columns: [
 | 
					    columns: [
 | 
				
			||||||
	{
 | 
						{
 | 
				
			||||||
	    header: gettext('Sync Job'),
 | 
						    header: gettext('Job ID'),
 | 
				
			||||||
	    dataIndex: 'id',
 | 
						    dataIndex: 'id',
 | 
				
			||||||
	    renderer: Ext.String.htmlEncode,
 | 
						    renderer: Ext.String.htmlEncode,
 | 
				
			||||||
	    flex: 2,
 | 
						    maxWidth: 220,
 | 
				
			||||||
 | 
						    minWidth: 75,
 | 
				
			||||||
 | 
						    flex: 1,
 | 
				
			||||||
	    sortable: true,
 | 
						    sortable: true,
 | 
				
			||||||
	    hidden: true,
 | 
					 | 
				
			||||||
	},
 | 
						},
 | 
				
			||||||
	{
 | 
						{
 | 
				
			||||||
	    header: gettext('Remote'),
 | 
						    header: gettext('Remote'),
 | 
				
			||||||
	    dataIndex: 'remote',
 | 
						    dataIndex: 'remote',
 | 
				
			||||||
	    flex: 2,
 | 
						    width: 120,
 | 
				
			||||||
	    sortable: true,
 | 
						    sortable: true,
 | 
				
			||||||
	},
 | 
						},
 | 
				
			||||||
	{
 | 
						{
 | 
				
			||||||
	    header: gettext('Remote Store'),
 | 
						    header: gettext('Remote Store'),
 | 
				
			||||||
	    dataIndex: 'remote-store',
 | 
						    dataIndex: 'remote-store',
 | 
				
			||||||
	    flex: 2,
 | 
						    width: 120,
 | 
				
			||||||
	    sortable: true,
 | 
						    sortable: true,
 | 
				
			||||||
	},
 | 
						},
 | 
				
			||||||
	{
 | 
						{
 | 
				
			||||||
	    header: gettext('Local Store'),
 | 
						    header: gettext('Local Store'),
 | 
				
			||||||
	    dataIndex: 'store',
 | 
						    dataIndex: 'store',
 | 
				
			||||||
	    flex: 2,
 | 
						    width: 120,
 | 
				
			||||||
	    sortable: true,
 | 
						    sortable: true,
 | 
				
			||||||
	},
 | 
						},
 | 
				
			||||||
	{
 | 
						{
 | 
				
			||||||
	    header: gettext('Schedule'),
 | 
						    header: gettext('Schedule'),
 | 
				
			||||||
	    dataIndex: 'schedule',
 | 
						    dataIndex: 'schedule',
 | 
				
			||||||
	    flex: 2,
 | 
						    maxWidth: 220,
 | 
				
			||||||
 | 
						    minWidth: 80,
 | 
				
			||||||
 | 
						    flex: 1,
 | 
				
			||||||
	    sortable: true,
 | 
						    sortable: true,
 | 
				
			||||||
	},
 | 
						},
 | 
				
			||||||
	{
 | 
					 | 
				
			||||||
	    header: gettext('Status'),
 | 
					 | 
				
			||||||
	    dataIndex: 'last-run-state',
 | 
					 | 
				
			||||||
	    renderer: 'render_sync_status',
 | 
					 | 
				
			||||||
	    flex: 4,
 | 
					 | 
				
			||||||
	},
 | 
					 | 
				
			||||||
	{
 | 
						{
 | 
				
			||||||
	    header: gettext('Last Sync'),
 | 
						    header: gettext('Last Sync'),
 | 
				
			||||||
	    dataIndex: 'last-run-endtime',
 | 
						    dataIndex: 'last-run-endtime',
 | 
				
			||||||
	    renderer: 'render_optional_timestamp',
 | 
						    renderer: 'render_optional_timestamp',
 | 
				
			||||||
	    flex: 3,
 | 
						    width: 150,
 | 
				
			||||||
	    sortable: true,
 | 
						    sortable: true,
 | 
				
			||||||
	},
 | 
						},
 | 
				
			||||||
	{
 | 
						{
 | 
				
			||||||
	    text: gettext('Duration'),
 | 
						    text: gettext('Duration'),
 | 
				
			||||||
	    dataIndex: 'duration',
 | 
						    dataIndex: 'duration',
 | 
				
			||||||
	    renderer: Proxmox.Utils.render_duration,
 | 
						    renderer: Proxmox.Utils.render_duration,
 | 
				
			||||||
	    flex: 2,
 | 
						    width: 80,
 | 
				
			||||||
 | 
						},
 | 
				
			||||||
 | 
						{
 | 
				
			||||||
 | 
						    header: gettext('Status'),
 | 
				
			||||||
 | 
						    dataIndex: 'last-run-state',
 | 
				
			||||||
 | 
						    renderer: 'render_sync_status',
 | 
				
			||||||
 | 
						    flex: 3,
 | 
				
			||||||
	},
 | 
						},
 | 
				
			||||||
	{
 | 
						{
 | 
				
			||||||
	    header: gettext('Next Run'),
 | 
						    header: gettext('Next Run'),
 | 
				
			||||||
	    dataIndex: 'next-run',
 | 
						    dataIndex: 'next-run',
 | 
				
			||||||
	    renderer: 'render_next_run',
 | 
						    renderer: 'render_next_run',
 | 
				
			||||||
	    flex: 3,
 | 
						    width: 150,
 | 
				
			||||||
	    sortable: true,
 | 
						    sortable: true,
 | 
				
			||||||
	},
 | 
						},
 | 
				
			||||||
	{
 | 
						{
 | 
				
			||||||
	    header: gettext('Comment'),
 | 
						    header: gettext('Comment'),
 | 
				
			||||||
	    dataIndex: 'comment',
 | 
						    dataIndex: 'comment',
 | 
				
			||||||
	    renderer: Ext.String.htmlEncode,
 | 
						    renderer: Ext.String.htmlEncode,
 | 
				
			||||||
	    flex: 4,
 | 
						    flex: 2,
 | 
				
			||||||
	    sortable: true,
 | 
						    sortable: true,
 | 
				
			||||||
	},
 | 
						},
 | 
				
			||||||
    ],
 | 
					    ],
 | 
				
			||||||
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user