#8078 enable multi choice #848

Merged
jorgep merged 17 commits from 8078-enableMultiSelection into dev 2024-11-05 10:20:05 +00:00
1 changed files with 2 additions and 2 deletions
Showing only changes of commit 77cb2d84be - Show all commits

View File

@ -335,8 +335,9 @@ function handleScroll() {
function handleSelection({ evt, added, rows: selectedRows }, rows) {
if (evt?.shiftKey && added) {
const rowIndex = selectedRows[0].$index;
Review

Solo selecciona 1, en el unico momento en que se pueden seleccionar más, es cuando le das al checkbox para seleccionarlas todas y evt es undefined. Además, quasar siempre añade la propiedad $index.

Solo selecciona 1, en el unico momento en que se pueden seleccionar más, es cuando le das al checkbox para seleccionarlas todas y **evt es undefined**. Además, quasar siempre añade la propiedad **$index**.
Review
Quasar o CrudModel? https://gitea.verdnatura.es/verdnatura/salix-front/src/commit/6248a4117d06e60ee4f141dcb5b65969c590bfbe/src/components/CrudModel.vue#L123 😏
selected.value.length = 0;

Podría hacer un filter, pero creo que mejor un for para no recorrerlas todas , en caso de que hayan muchos registros.

Podría hacer un filter, pero creo que mejor un for para no recorrerlas todas , en caso de que hayan muchos registros.
for (const row of rows) {
if (row.$index > rowIndex) break;
if (row.$index == rowIndex) break;
selected.value.push(row);
Review

Así se comprueba si la fila ya está seleccionada y no se recorren todas las filas.

Así se comprueba si la fila ya está seleccionada y no se recorren todas las filas.
}
}
@ -440,7 +441,6 @@ function handleSelection({ evt, added, rows: selectedRows }, rows) {
:virtual-scroll="isTableMode"
@virtual-scroll="handleScroll"
@row-click="(_, row) => rowClickFunction && rowClickFunction(row)"
@update:selected="emit('update:selected', $event)"
@selection="(details) => handleSelection(details, rows)"
>
<template #top-left v-if="!$props.withoutHeader">