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

#7353 fine tunning monitors #624

Merged
jorgep merged 34 commits from 7353-fineTunningMonitor into dev 2024-09-02 07:33:43 +00:00
Conversation 1 Commits 34 Files Changed 22 +11 -17
1 changed files with 11 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit ed2167e199 - Show all commits

28
src/pages/Monitor/MonitorClients.vue
View File

@ -18,17 +18,13 @@ const filter = computed(() => {
const obj = {}; const obj = {};
const formatFrom = setHours(from.value, 'from'); const formatFrom = setHours(from.value, 'from');
const formatTo = setHours(to.value, 'to'); const formatTo = setHours(to.value, 'to');
let stamp;
jorgep marked this conversation as resolved Outdated
jsegarra commented 2024-08-30 21:47:55 +00:00
Outdated
Review
Copy Link

Veo mucho Object.assign.
Crees que podemos dejar esto debajo del if...else..if y que el valor de v.stamp sea una variable?

Veo mucho Object.assign. Crees que podemos dejar esto debajo del if...else..if y que el valor de v.stamp sea una variable?
if (!formatFrom && formatTo) if (!formatFrom && formatTo) stamp = { lte: formatTo };
Object.assign(obj, { where: { 'v.stamp': { lte: formatTo } } }); else if (formatFrom && !formatTo) stamp = { gte: formatFrom };
else if (formatFrom && !formatTo) else if (formatFrom && formatTo) stamp = { between: [formatFrom, formatTo] };
Object.assign(obj, { where: { 'v.stamp': { gte: formatFrom } } });
else if (formatFrom && formatTo)
Object.assign(obj, {
where: { 'v.stamp': { between: [formatFrom, formatTo] } },
});
return obj; return Object.assign(obj, { where: { 'v.stamp': stamp } });
}); });
function exprBuilder(param, value) { function exprBuilder(param, value) {
@ -84,6 +80,7 @@ const columns = computed(() => [
useLike: false, useLike: false,
}, },
}, },
columnClass: 'no-padding',
}, },
{ {
label: t('salesClientsTable.client'), label: t('salesClientsTable.client'),
@ -102,6 +99,7 @@ const columns = computed(() => [
sortBy: 'name ASC', sortBy: 'name ASC',
}, },
}, },
columnClass: 'no-padding',
}, },
]); ]);
</script> </script>
@ -131,16 +129,12 @@ const columns = computed(() => [
</VnRow> </VnRow>
jorgep marked this conversation as resolved Outdated
jsegarra commented 2024-08-28 18:49:50 +00:00
Outdated
Review
Copy Link

Traducir

Traducir
</template> </template>
jorgep marked this conversation as resolved Outdated
jsegarra commented 2024-08-28 18:49:55 +00:00
Outdated
Review
Copy Link

Traducir

Traducir
<template #column-salesPersonFk="{ row }"> <template #column-salesPersonFk="{ row }">
<QTd class="no-padding"> <span class="link" :title="row.salesPerson" v-text="row.salesPerson" />
<span class="link" :title="row.salesPerson" v-text="row.salesPerson" /> <WorkerDescriptorProxy :id="row.salesPersonFk" dense />
<WorkerDescriptorProxy :id="row.salesPersonFk" dense />
</QTd>
</template> </template>
jorgep marked this conversation as resolved Outdated
jsegarra commented 2024-08-30 21:48:52 +00:00
Outdated
Review
Copy Link

Revisa porque creo que le sobre el Qtd y le falta el @click.stop
O pones Qtd por el .no-padding?

Revisa porque creo que le sobre el Qtd y le falta el @click.stop O pones Qtd por el .no-padding?
jorgep commented 2024-09-02 07:11:29 +00:00
Outdated
Review
Copy Link

Esta tabla no redirige a ningún sitio, no le hace falta el click.stop. El qTd lo quito.

Esta tabla no redirige a ningún sitio, no le hace falta el click.stop. El qTd lo quito.
<template #column-clientFk="{ row }"> <template #column-clientFk="{ row }">
<QTd class="no-padding"> <span class="link" :title="row.clientName" v-text="row.clientName" />
<span class="link" :title="row.clientName" v-text="row.clientName" /> <CustomerDescriptorProxy :id="row.clientFk" />
<CustomerDescriptorProxy :id="row.clientFk" />
</QTd>
</template> </template>
</VnTable> </VnTable>
</template> </template>
jsegarra commented 2024-08-28 18:52:11 +00:00
Outdated
Review
Copy Link

El texto no aparece completo
"Bruce Way..." es lo que se ve

El texto no aparece completo "Bruce Way..." es lo que se ve
jorgep commented 2024-08-29 10:30:44 +00:00
Outdated
Review
Copy Link

Le he quitado el padding, con el espacio que hay poco más se puede hacer. la tabla tiene el ancho que toca. Ahora los nombre de esa longitud y un poco más se ven.

Le he quitado el padding, con el espacio que hay poco más se puede hacer. la tabla tiene el ancho que toca. Ahora los nombre de esa longitud y un poco más se ven.