Re: [PATCH manager 2/2] ui: vCPU flag selector: re-enable sortable columns

[Arthur Bied-Charreton <a.bied-charreton@proxmox.com>]

On Mon, May 25, 2026 at 03:15:26PM +0200, Thomas Lamprecht wrote:
> The preceding commit replaced the radio widgets, whose inputs went
> stale in reordered rows, with segmented buttons that re-sync on widget
> attach. As that was the only reason sorting was disabled as a
> stop-gap, we can drop that again.
> 
> Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
> ---
>  www/manager6/form/VMCPUFlagSelector.js | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/www/manager6/form/VMCPUFlagSelector.js b/www/manager6/form/VMCPUFlagSelector.js
> index 5972ee678..9cd076166 100644
> --- a/www/manager6/form/VMCPUFlagSelector.js
> +++ b/www/manager6/form/VMCPUFlagSelector.js
> @@ -4,9 +4,6 @@ Ext.define('PVE.form.VMCPUFlagSelector', {
>  
>      bufferedRenderer: false,
>  
> -    // FIXME: stop-gap; sorting caused glitches with the inputs of radio widgets in moved rows.
> -    sortableColumns: false,
> -
>      mixins: {
>          field: 'Ext.form.field.Field',
>      },
> -- 
> 2.47.3
> 
Did you intend to make the flags sortable by state? Ext.js does not seem
to enable sorting by default on widgetcolumns (at least the examples in
the docs [0] are also not sortable).

If yes (I think such a sort would be helpful to see which flags are
enabled), we would need to add sortable: true to the "Value" column. 

[0] https://docs.sencha.com/extjs/7.0.0/guides/components/widgets_widgets_columns.html