[pve-devel] applied: [PATCH manager v2 1/1] ui: improve form/MDevSelector

[Wolfgang Bumiller]

applied, thanks

On Fri, Aug 12, 2022 at 11:29:51AM +0200, Dominik Csapak wrote:
> by
> * showing the (optional) name in front of the type
> * making the 'availble' column a bit narrower
> * enabling 'cellWrap' for the description
> * making the dropdown a bit wider (so all the information can fit)
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
>  www/manager6/form/MDevSelector.js | 12 ++++++++++--
>  1 file changed, 10 insertions(+), 2 deletions(-)
> 
> diff --git a/www/manager6/form/MDevSelector.js b/www/manager6/form/MDevSelector.js
> index 8ee73c0c..0a157813 100644
> --- a/www/manager6/form/MDevSelector.js
> +++ b/www/manager6/form/MDevSelector.js
> @@ -16,21 +16,29 @@ Ext.define('PVE.form.MDevSelector', {
>      valueField: 'type',
>      displayField: 'type',
>      listConfig: {
> +	width: 550,
>  	columns: [
>  	    {
>  		header: gettext('Type'),
>  		dataIndex: 'type',
> +		renderer: function(value, md, rec) {
> +		    if (rec.data.name !== undefined) {
> +			return `${rec.data.name} (${value})`;
> +		    }
> +		    return value;
> +		},
>  		flex: 1,
>  	    },
>  	    {
> -		header: gettext('Available'),
> +		header: gettext('Avail.'),
>  		dataIndex: 'available',
> -		width: 80,
> +		width: 60,
>  	    },
>  	    {
>  		header: gettext('Description'),
>  		dataIndex: 'description',
>  		flex: 1,
> +		cellWrap: true,
>  		renderer: function(value) {
>  		    if (!value) {
>  			return '';
> -- 
> 2.30.2