feat: add VnMultiCheck component for multi-selection functionality #1734

Merged
jsegarra merged 9 commits from vnTable_multi_check into dev 2025-04-23 17:53:07 +00:00
Member

Hay un ejemplo hecho en Clientes>Notificaciones

Hay un ejemplo hecho en Clientes>Notificaciones
jsegarra added 1 commit 2025-04-22 15:43:46 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
b13bcea331
feat: add VnMultiCheck component for multi-selection functionality
jsegarra changed title from feat: add VnMultiCheck component for multi-selection functionality to WIP: feat: add VnMultiCheck component for multi-selection functionality 2025-04-22 15:43:53 +00:00
jsegarra requested review from pablone 2025-04-22 15:44:04 +00:00
jsegarra added 2 commits 2025-04-22 16:03:14 +00:00
jsegarra changed title from WIP: feat: add VnMultiCheck component for multi-selection functionality to feat: add VnMultiCheck component for multi-selection functionality 2025-04-22 16:03:22 +00:00
pablone requested changes 2025-04-23 11:38:14 +00:00
pablone left a comment
Member

Te fallan un par de test de itemFixedPrice, ns si són intermitentes

Te fallan un par de test de itemFixedPrice, ns si són intermitentes
jsegarra added 1 commit 2025-04-23 13:41:47 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
af8909177e
Merge branch 'dev' into vnTable_multi_check
jsegarra added 5 commits 2025-04-23 17:25:19 +00:00
jsegarra requested review from pablone 2025-04-23 17:53:00 +00:00
jsegarra merged commit 2c8d3ac0e0 into dev 2025-04-23 17:53:07 +00:00
pablone requested changes 2025-04-24 11:09:23 +00:00
@ -113,6 +114,10 @@ const $props = defineProps({
type: Object,
default: () => ({}),
},
multiCheck: {
Member

Esto es subjetivo: yo le pondria de nombre a la variable backMultiCheck

Esto es subjetivo: yo le pondria de nombre a la variable backMultiCheck
@ -662,3 +685,3 @@
v-bind="$attrs"
:class="$attrs['class'] ?? 'q-px-md'"
:limit="$attrs['limit'] ?? 100"
:limit="$attrs['limit'] ?? 10"
Member

???

???
Author
Member

Corregido a posterior

Corregido a posterior
jsegarra marked this conversation as resolved
@ -700,6 +723,18 @@ const rowCtrlClickFunction = computed(() => {
:hide-selected-banner="true"
:data-cy
>
<template #header-selection>
Member

condicional que sea intencional activarlo ya que solo funcionara con ciertos backs construidos con loopback

condicional que sea intencional activarlo ya que solo funcionara con ciertos backs construidos con loopback
Author
Member

Delegamos la responsabilidad en el componente, y segun si la prop es true o false se mostrará el icono

Delegamos la responsabilidad en el componente, y segun si la prop es true o false se mostrará el icono
@ -701,2 +724,4 @@
:data-cy
>
<template #header-selection>
{{ $attrs['url'] }}
Member

borrar

borrar
Author
Member

corregido a posterior

corregido a posterior
jsegarra marked this conversation as resolved
@ -0,0 +17,4 @@
type: String,
default: null,
required: true,
},
Member

required: true y default: null no tiene sentido

required: true y default: null no tiene sentido
Author
Member

cierto

cierto
jsegarra marked this conversation as resolved
@ -0,0 +37,4 @@
.then(({ data }) => {
rows.value = data;
})
.catch(console.error);
Member

transparente para el usuario si falla

transparente para el usuario si falla
jsegarra marked this conversation as resolved
@ -0,0 +56,4 @@
<QMenu anchor="bottom right" self="top right">
<QList>
<QItem v-ripple clickable @click="$emit('select:all', toRaw(rows))">
{{ t('Select all', { rows: rows.length }) }}
Member

v-text

v-text
jsegarra marked this conversation as resolved
@ -0,0 +58,4 @@
<QItem v-ripple clickable @click="$emit('select:all', toRaw(rows))">
{{ t('Select all', { rows: rows.length }) }}
</QItem>
<slot name="more-options"></slot>
Member

te sobra 1 etiqueta

te sobra 1 etiqueta
Author
Member

He dejado la opcion por si quieres poner a demanda mas opciones

He dejado la opcion por si quieres poner a demanda mas opciones
@ -0,0 +68,4 @@
en:
Select all: 'Select all ({rows})'
fr:
Select all: 'Sélectionner tout ({rows})'
Member

de momento solo hacesmo español/ingles

de momento solo hacesmo español/ingles
jsegarra marked this conversation as resolved
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix-front#1734
No description provided.