[pbs-devel] [PATCH widget-toolkit 5/6] fix #5379: panel: AuthView: add column displaying whether the realm is default

Lukas Wagner l.wagner at proxmox.com
Wed Aug 7 11:24:10 CEST 2024


On  2024-07-16 15:45, Christoph Heiss wrote:
> Signed-off-by: Christoph Heiss <c.heiss at proxmox.com>
> ---
>  src/panel/AuthView.js | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/src/panel/AuthView.js b/src/panel/AuthView.js
> index 944a812..128d863 100644
> --- a/src/panel/AuthView.js
> +++ b/src/panel/AuthView.js
> @@ -25,6 +25,13 @@ Ext.define('Proxmox.panel.AuthView', {
>  	    sortable: true,
>  	    dataIndex: 'type',
>  	},
> +	{
> +	    header: gettext('Default'),
> +	    width: 80,
> +	    sortable: true,
> +	    dataIndex: 'default',
> +	    renderer: isDefault => isDefault ? Proxmox.Utils.renderEnabledIcon(true) : '',
> +	},
>  	{
>  	    header: gettext('Comment'),
>  	    sortable: false,

I think the check mark should be centered in the column, see e.g. the enabled
column for notification targets/matchers. Looks a bit more consistent then 😄.

-- 
- Lukas




More information about the pbs-devel mailing list