verdnatura/salix-front
verdnatura
/
salix-front
1
2
Fork 1
Code Issues Pull Requests 62 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 +2 -2
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 86e6bef90f - Show all commits

4
test/vitest/__tests__/components/VnTable.spec.js
View File

@ -20,12 +20,12 @@ describe('VnTable', () => {
describe('handleSelection()', () => {
const rows = [{ $index: 0 }, { $index: 1 }, { $index: 2 }];
const selectedRows = [{ $index: 1 }];
it('should add rows to selected when shift key is pressed and rows are added', () => {
it('should add rows to selected when shift key is pressed and rows are added except last one', () => {
vm.handleSelection(
{ evt: { shiftKey: true }, added: true, rows: selectedRows },
rows
);
expect(vm.selected).toEqual([{ $index: 0 }, { $index: 1 }]);
expect(vm.selected).toEqual([{ $index: 0 }]);
jorgep commented 2024-11-05 09:41:20 +00:00
Review
Copy Link

El de index 1 no lo pongo xq se añade desde el evt update:selected no desde esta fn.

El de index 1 no lo pongo xq se añade desde el evt update:selected no desde esta fn.
});
it('should not add rows to selected when shift key is not pressed', () => {
jorgep commented 2024-10-28 14:42:12 +00:00
Review
Copy Link

No se añade dentro de la fn, pero si que se selecciona la fila en el evt @update:selected , se ejecuta después de @selection

No se añade dentro de la fn, pero si que se selecciona la fila en el evt **@update:selected** , se ejecuta después de **@selection**