verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 86 Releases Wiki Activity

#8078 enable multi choice #848

Merged
jorgep merged 17 commits from 8078-enableMultiSelection into dev 2024-11-05 10:20:05 +00:00
Conversation 2 Commits 17 Files Changed 5 +5 -2
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit e39f85ff4b - Show all commits

5
src/components/VnTable/VnTable.vue
View File

@ -335,10 +335,13 @@ function handleScroll() {
function handleSelection({ evt, added, rows: selectedRows }, rows) { function handleSelection({ evt, added, rows: selectedRows }, rows) {
if (evt?.shiftKey && added) { if (evt?.shiftKey && added) {
const rowIndex = selectedRows[0].$index; const rowIndex = selectedRows[0].$index;
jorgep commented 2024-10-28 11:08:27 +00:00
Review
Copy Link

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**.
alexm commented 2024-10-29 06:45:44 +00:00
Review
Copy Link

Quasar o CrudModel? 6248a4117d/src/components/CrudModel.vue (L123) 😏

Quasar o CrudModel? https://gitea.verdnatura.es/verdnatura/salix-front/src/commit/6248a4117d06e60ee4f141dcb5b65969c590bfbe/src/components/CrudModel.vue#L123 😏
🎉 1
selected.value.length = 0; const selectedIndexes = new Set(selected.value.map((row) => row.$index));
jorgep commented 2024-10-28 11:09:00 +00:00
Outdated
Review
Copy Link

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) { for (const row of rows) {
if (row.$index == rowIndex) break; if (row.$index == rowIndex) break;
if (!selectedIndexes.has(row.$index)) {
jorgep commented 2024-10-29 09:06:14 +00:00
Review
Copy Link

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.
selected.value.push(row); selected.value.push(row);
selectedIndexes.add(row.$index);
}
} }
} }
} }