[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