[pve-devel] applied: [PATCH widget-toolkit] improve multiselect behaviour for checkboxmodel

Thomas Lamprecht t.lamprecht at proxmox.com
Fri Oct 23 13:28:33 CEST 2020


On 22.04.20 09:26, Dominik Csapak wrote:
> when using the mode 'MULTI' for the checkbox selection model,
> clicking a row would deselect all others, only clicking exactly on the
> checkbox would select multiple rows
> 
> this override makes the whole cell of the checkbox behave like the
> checkbox. This minimizes misclicks from the user.
> 
> Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
> ---
> btw. the extjs docs lie, the default is not
> 'x-grid-row-checker' but '.x-grid-row-checker'
> so when setting a different class the '.' is not optional
> 20 minutes of my life i will not get back...
> 
>  Toolkit.js | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
>

better late than never, I guess :)

applied, thanks!

FYI: had to use apply it with 3-way merge, as there where some context changes
(possibly the ones from eslint fixes).

> Applying: improve multiselect behaviour for checkboxmodel
> Using index info to reconstruct a base tree...
> A       Toolkit.js
> Falling back to patching base and 3-way merge...
> Auto-merging src/Toolkit.js





More information about the pve-devel mailing list