#7283 finish item card sections #588

Merged
jorgep merged 24 commits from 7283-itemSectionsMigration into dev 2024-08-12 12:08:46 +00:00
Member
No description provided.
jorgep added 1 commit 2024-08-02 10:50:13 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
bc4ee5c8a5
fix: refs #7283 basicData locale
jorgep added 1 commit 2024-08-02 10:51:02 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
27e7d30158
fix: refs #7283 basicData locale
jorgep added 1 commit 2024-08-02 12:31:44 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
6f771c7ae6
fix: refs #7283 basicData locale
jorgep added 1 commit 2024-08-05 11:01:31 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
16cfd84fab
fix: refs #7283 itemTags & VnImg
jorgep added 2 commits 2024-08-05 15:13:20 +00:00
jorgep added 1 commit 2024-08-05 15:14:22 +00:00
jorgep added 1 commit 2024-08-06 06:47:15 +00:00
jorgep added 1 commit 2024-08-06 06:56:15 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
50468d22cc
fix: refs #7283 min-width vnImg
jorgep added 1 commit 2024-08-06 07:12:14 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
90073a0a68
fix: refs #7283 locale
jorgep added 1 commit 2024-08-06 07:51:26 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
1411833a4c
chore: refs #7283 fix e2e logout
jorgep changed title from WIP: #7283 finish item card sections to #7283 finish item card sections 2024-08-06 07:54:44 +00:00
jorgep reviewed 2024-08-06 07:55:10 +00:00
@ -89,2 +75,2 @@
/>
</QAvatar>
<QBtn class="q-pa-none" rounded dense flat no-wrap id="user">
<VnAvatar
Author
Member

Es la foto de un usuario

Es la foto de un usuario
jorgep reviewed 2024-08-06 07:55:40 +00:00
@ -157,3 +156,1 @@
<VnImg :id="user.id" collection="user" size="160x160" />
</QAvatar>
<VnAvatar
Author
Member

Es la foto de un usuario.

Es la foto de un usuario.
jorgep reviewed 2024-08-06 07:55:56 +00:00
@ -32,3 +32,3 @@
<QCard class="card shadow-6">
<div class="img-wrapper">
<VnImg :id="item.id" zoom-size="lg" class="image" />
<VnImg :id="item.id" class="image" />
Author
Member

Ese lg no hacía nada.

Ese lg no hacía nada.
jorgep reviewed 2024-08-06 07:57:04 +00:00
@ -9,3 +9,2 @@
description: { type: String, default: null },
size: { type: String, default: null },
title: { type: String, default: null },
title: { type: [String, Boolean], default: null },
Author
Member

No siempre se quiere poner un title, ej. cuando hay un desplegable.

No siempre se quiere poner un title, ej. cuando hay un desplegable.
Owner

se puede directamente no definir la propiedad? así no es necesario el boolean?

se puede directamente no definir la propiedad? así no es necesario el boolean?
jorgep marked this conversation as resolved
jorgep reviewed 2024-08-06 07:57:11 +00:00
@ -10,2 +10,2 @@
size: { type: String, default: null },
title: { type: String, default: null },
title: { type: [String, Boolean], default: null },
color: { type: String, default: null },
Author
Member

Habilita

Habilita
jorgep reviewed 2024-08-06 07:58:11 +00:00
@ -12,3 +14,3 @@
default: 'catalog',
},
size: {
resolution: {
Author
Member

Realmente no se le estaba indicando el tamaño, solo la resolución

Realmente no se le estaba indicando el tamaño, solo la resolución
jorgep reviewed 2024-08-06 07:58:31 +00:00
@ -21,1 +22,3 @@
default: 'lg',
default: null,
},
zoom: {
Author
Member

No siempre se quiere hacer zoom, ej. contador de carros.

No siempre se quiere hacer zoom, ej. contador de carros.
jorgep reviewed 2024-08-06 07:59:08 +00:00
@ -4,3 +2,2 @@
<template>
<div class="vn-row q-gutter-md q-mb-md" :class="{ wrap }">
<slot></slot>
<div class="vn-row q-gutter-md q-mb-md">
Author
Member

wrap no hace nada, se deshicieron los cambios.

wrap no hace nada, se deshicieron los cambios.
jorgep reviewed 2024-08-06 08:00:00 +00:00
@ -7,3 +5,3 @@
</div>
</template>
<style lang="scss" scopped>
<style lang="scss" scoped>
Author
Member

scopped lleva solo una p, antes era un style global. Con :deep y scoped funciona correctamente

scopped lleva solo una p, antes era un style global. Con **:deep** y **scoped** funciona correctamente
jorgep reviewed 2024-08-06 08:00:56 +00:00
@ -74,3 +65,3 @@
<VnRow>
<VnSelect
:label="t('basicData.type')"
:label="t('itemBasicData.type')"
Author
Member

hay otro archivo donde ya se usa la clave basicData, por lo que daba conflicto

hay otro archivo donde ya se usa la clave basicData, por lo que daba conflicto
jorgep added the
CR / Tests passed
label 2024-08-06 08:05:14 +00:00
jorgep requested review from jgallego 2024-08-06 08:05:20 +00:00
jorgep reviewed 2024-08-06 08:06:04 +00:00
@ -43,3 +43,3 @@
};
const handleTagSelected = (rows, index, tag) => {
const handleTagSelected = (rows, index, tagId) => {
Author
Member

Ahora se recoge correctamente el tag

Ahora se recoge correctamente el tag
jorgep reviewed 2024-08-06 08:06:31 +00:00
@ -110,4 +110,0 @@
<VnRow
v-for="(row, index) in rows"
:key="index"
class="row q-gutter-md q-mb-md"
Author
Member

No hacía nada.

No hacía nada.
jorgep reviewed 2024-08-06 08:07:16 +00:00
@ -120,3 +117,3 @@
hide-selected
@update:model-value="
($event) => handleTagSelected(rows, index, $event)
(val) => handleTagSelected(rows, index, val)
Author
Member

más descriptivo, es un evt/fn personalizado.

más descriptivo, es un evt/fn personalizado.
jorgep reviewed 2024-08-06 08:07:50 +00:00
@ -156,3 +152,3 @@
:rules="validate('itemTag.priority')"
/>
<div class="col-1 row justify-center items-center">
<div class="row justify-center items-center" style="flex: 0">
Author
Member

El basurero se alinea a la derecha y ocupa el mínimo espacio posible.

El basurero se alinea a la derecha y ocupa el mínimo espacio posible.
jorgep reviewed 2024-08-06 08:08:46 +00:00
@ -177,6 +173,7 @@ const insertTag = (rows) => {
color="primary"
name="add"
size="sm"
style="flex: 0"
Author
Member

El area "clicable" era bastante más ancha que el propio icono. ahora ocupa solo el espacio del icono.

El area "clicable" era bastante más ancha que el propio icono. ahora ocupa solo el espacio del icono.
jorgep reviewed 2024-08-06 08:09:40 +00:00
@ -65,3 +65,3 @@
<QItem v-for="(props, name) in counters" :key="name" class="col-6">
<QItemSection>
<VnImg :id="props.id" width="130px" @click="handleEvent(name, 'add')" />
<VnImg
Author
Member

si haces click para sumar 1 también te hace zoom. No tenía sentido.

si haces click para sumar 1 también te hace zoom. No tenía sentido.
alexm approved these changes 2024-08-07 08:13:43 +00:00
Dismissed
jorgep added 1 commit 2024-08-07 08:17:20 +00:00
jorgep added 1 commit 2024-08-07 13:36:36 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
88dffbce8a
Merge branch 'dev' into 7283-itemSectionsMigration
jorgep requested review from alexm 2024-08-07 13:37:42 +00:00
alexm approved these changes 2024-08-07 13:38:12 +00:00
Dismissed
jorgep added 1 commit 2024-08-08 06:57:55 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
0fa0ddec7e
Merge branch 'dev' into 7283-itemSectionsMigration
jorgep added 2 commits 2024-08-09 10:13:29 +00:00
jorgep added 1 commit 2024-08-09 10:34:14 +00:00
jorgep added 1 commit 2024-08-09 13:29:45 +00:00
jorgep added 1 commit 2024-08-09 13:34:15 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
f17ca4fe30
fix: refs #7283 itemLastEntries filter
jorgep dismissed alexm’s review 2024-08-09 13:34:15 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

jorgep added 1 commit 2024-08-09 13:42:36 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
6b80c52b71
chore: refs #7283 drop import
jgallego requested changes 2024-08-12 05:36:01 +00:00
Dismissed
@ -160,0 +156,4 @@
<VnAvatar
:worker-id="user.id"
:title="user.name"
size="80px"
Owner

en otros sitios se usa size="md" posible usar los genericos.
Pregunta: no he encontrado donde esta en nuestro caso md a cuantos px equivale, donde se está?

en otros sitios se usa size="md" posible usar los genericos. Pregunta: no he encontrado donde esta en nuestro caso md a cuantos px equivale, donde se está?
Author
Member

No lo tenemos definido, es nativo de quasar. He probado a poner xl pero se queda muy pequeño. Creo un tamaño personalizado para q-avatar[size="xxxl"] que sea 80px?

No lo tenemos definido, es nativo de quasar. He probado a poner **xl** pero se queda muy pequeño. Creo un tamaño personalizado para q-avatar[size="xxxl"] que sea 80px?
Owner

si

si
@ -107,1 +100,3 @@
</QAvatar>
<VnAvatar
:worker-id="data.salesPersonFk"
color="orange"
Owner

los colores estan en quasar.variables.scss
usa las variables, de paso puedes reemplazar los orange que encuentres

los colores estan en quasar.variables.scss usa las variables, de paso puedes reemplazar los orange que encuentres
jorgep marked this conversation as resolved
jorgep requested review from jgallego 2024-08-12 07:56:56 +00:00
jorgep added 1 commit 2024-08-12 08:18:17 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
f5ffe2a07b
refactor: refs #7283 update avatar size and color
jorgep added 1 commit 2024-08-12 09:00:09 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
239e796d2e
chore: refs #7283 update VnAvatar title handling
jorgep added 1 commit 2024-08-12 09:01:36 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
9708214462
chore: refs #7283 drop array types
jorgep added 1 commit 2024-08-12 10:51:42 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
2cf669b479
Merge branch 'dev' into 7283-itemSectionsMigration
jgallego approved these changes 2024-08-12 11:48:36 +00:00
jorgep merged commit 59262019f8 into dev 2024-08-12 12:08:46 +00:00
jorgep deleted branch 7283-itemSectionsMigration 2024-08-12 12:08:46 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 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#588
No description provided.