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

#6911 save on enter #207

Merged
jsegarra merged 50 commits from 6911-saveOnEnter into dev 2024-06-07 07:20:48 +00:00
Conversation 6 Commits 50 Files Changed 12 +276 -239
jorgep commented 2024-02-26 13:37:34 +00:00
Member
Copy Link
No description provided.
jorgep added 1 commit 2024-02-27 08:32:47 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
2c4676780e fix: refs #6911 add icons, fix invoiceIn & save on enter
jorgep added 1 commit 2024-02-27 09:40:52 +00:00
jorgep added 1 commit 2024-02-27 10:29:34 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
ba1dea32a5 fix: refs #6911 save onKeyup enter
jorgep requested review from jgallego 2024-02-27 10:30:21 +00:00
jorgep requested review from jsegarra 2024-02-27 10:30:21 +00:00
jorgep changed title from WIP: #6911 save on enter to #6911 save on enter 2024-02-27 10:30:29 +00:00
jorgep commented 2024-02-27 10:33:27 +00:00
Author
Member
Copy Link

@jgallego @jsegarra iconos añadidos y correcciones en invoiceIn tras lo hablado con @jgallego, además de la tarea principal.

@jgallego @jsegarra iconos añadidos y correcciones en invoiceIn tras lo hablado con @jgallego, además de la tarea principal.
jgallego requested changes 2024-02-28 07:14:19 +00:00
Dismissed
src/components/FilterTravelForm.vue Outdated
@ -141,1 +141,3 @@
<QForm @submit="fetchResults()" class="all-pointer-events">
<QForm
@submit="fetchResults()"
@keyup.enter="fetchResults()"
jgallego commented 2024-02-28 07:14:08 +00:00
Owner
Copy Link

veo muchos @keyup.enter=
si todos los QForm los vamos a querer así.. @jsegarra no hay forma de hacerlo por defecto así?

veo muchos @keyup.enter= si todos los QForm los vamos a querer así.. @jsegarra no hay forma de hacerlo por defecto así?
jorgep commented 2024-02-29 11:02:31 +00:00
Author
Member
Copy Link

Propone sustituir en todos los sitios donde haya QForm por FormModel. Lo hago en esta tarea?

Propone sustituir en todos los sitios donde haya QForm por FormModel. Lo hago en esta tarea?
jorgep added 1 commit 2024-02-28 07:41:52 +00:00
jorgep added the
CR / Tests passed
 label 2024-02-28 13:05:43 +00:00
jorgep requested review from jgallego 2024-02-29 11:39:21 +00:00
jgallego approved these changes 2024-02-29 14:03:58 +00:00
Dismissed
jgallego left a comment
Owner
Copy Link

que @jsegarra valore si es posible en el refactor de QTable que el keyup vaya implicito en el form

que @jsegarra valore si es posible en el refactor de QTable que el keyup vaya implicito en el form
👀 1
jorgep added 1 commit 2024-02-29 14:28:43 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
b0cc7283b6 Merge branch 'dev' into 6911-saveOnEnter
jorgep added 2 commits 2024-03-01 14:02:52 +00:00
jorgep dismissed jgallego’s review 2024-03-01 14:02:52 +00:00
Reason:

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

jsegarra added 4 commits 2024-03-04 10:15:39 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
58dc4c253a refs #6911 perf: use QForm instead classList
jsegarra added 1 commit 2024-03-04 10:37:25 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
6bc8cdc0c3 refs #6911 feat: reaneme method and remove keyup
jsegarra added 1 commit 2024-03-04 10:38:56 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
ef979f05a3 refs #6911 feat: reaneme method and remove keyup
jorgep added 2 commits 2024-03-04 12:41:39 +00:00
jsegarra added 1 commit 2024-03-22 06:35:48 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
7ae05a3e52 refs #6911: perf KeyCode is deprecated
jorgep added 3 commits 2024-03-27 10:49:58 +00:00
jorgep added 1 commit 2024-03-27 10:53:30 +00:00
jcasado added 1 commit 2024-04-29 07:40:37 +00:00
jcasado added 1 commit 2024-04-30 06:14:50 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
d8d805a0ca refs #6911 changesSaveOnEnter
jcasado added 1 commit 2024-04-30 06:15:42 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
38f25eb02d Merge branch 'dev' into 6911-saveOnEnter
jcasado added 1 commit 2024-05-02 05:28:25 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
e44e182fec Merge branch 'dev' into 6911-saveOnEnter
jcasado added 1 commit 2024-05-03 05:22:02 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
bdfcba5c3e Merge branch 'dev' into 6911-saveOnEnter
jcasado requested review from alexm 2024-05-03 05:24:47 +00:00
alexm requested changes 2024-05-06 05:23:24 +00:00
Dismissed
alexm left a comment
Member
Copy Link

He probado el codigo y no he visto que funcione
He probado uno que este modificado aqui como http://localhost:9000/#/invoice-in/4/basic-data
Y uno que tiene FormModel (dentro tiene QForm) http://localhost:9000/#/claim/1/basic-data

En este ultimo tambien he probado a cambiarle el nombre a la funcion save de FormModel por onSubmitpero tampoco ha funcionado

He probado el codigo y no he visto que funcione He probado uno que este modificado aqui como `http://localhost:9000/#/invoice-in/4/basic-data` Y uno que tiene FormModel (dentro tiene QForm) `http://localhost:9000/#/claim/1/basic-data` En este ultimo tambien he probado a cambiarle el nombre a la funcion `save` de FormModel por `onSubmit`pero tampoco ha funcionado
jcasado added 1 commit 2024-05-06 11:24:50 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
fc94454cfd Merge branch 'dev' into 6911-saveOnEnter
jcasado added 1 commit 2024-05-09 06:04:29 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
b1f25266aa Merge branch 'dev' into 6911-saveOnEnter
jcasado added 1 commit 2024-05-09 07:28:05 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
9e7301d32e Merge branch 'dev' into 6911-saveOnEnter
jcasado added 1 commit 2024-05-09 11:17:40 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
8128e22645 Merge branch 'dev' into 6911-saveOnEnter
jsegarra added 1 commit 2024-05-09 15:48:53 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
a15530dba2 fix: replace key value
jsegarra requested review from alexm 2024-05-09 15:49:12 +00:00
jsegarra commented 2024-05-09 15:50:34 +00:00
Member
Copy Link

He probado el codigo y no he visto que funcione
He probado uno que este modificado aqui como http://localhost:9000/#/invoice-in/4/basic-data
Y uno que tiene FormModel (dentro tiene QForm) http://localhost:9000/#/claim/1/basic-data

En este ultimo tambien he probado a cambiarle el nombre a la funcion save de FormModel por onSubmitpero tampoco ha funcionado

Es correcto que no vaya, se cambio keyCode === 13 por key==='ENTER', y la comparación siempre daba false porque es 'Enter'
Ya he cambiado la comparación, asi que ya puedes probar.

> He probado el codigo y no he visto que funcione > He probado uno que este modificado aqui como `http://localhost:9000/#/invoice-in/4/basic-data` > Y uno que tiene FormModel (dentro tiene QForm) `http://localhost:9000/#/claim/1/basic-data` > > En este ultimo tambien he probado a cambiarle el nombre a la funcion `save` de FormModel por `onSubmit`pero tampoco ha funcionado Es correcto que no vaya, se cambio keyCode === 13 por key==='ENTER', y la comparación siempre daba false porque es 'Enter' Ya he cambiado la comparación, asi que ya puedes probar.
jgallego approved these changes 2024-05-10 08:50:03 +00:00
Dismissed
alexm commented 2024-05-13 05:35:06 +00:00
Member
Copy Link

Hay casos (sobre todo con QTextArea) en los que al hacer "enter" se quería un salto de linea:
Como en http://localhost:9000/#/claim/1/notes
image

En estos casos como se solucionaría?
Cambiamos para que el salto de linea sea con "Ctrl + Enter", más al estilo RocketChat?
@jgallego

Hay casos (sobre todo con QTextArea) en los que al hacer "enter" se quería un salto de linea: Como en `http://localhost:9000/#/claim/1/notes` ![image](/attachments/4a84957c-ed34-48f3-ac9e-6043065688e1) En estos casos como se solucionaría? Cambiamos para que el salto de linea sea con "Ctrl + Enter", más al estilo RocketChat? @jgallego
image.png
99 KiB
👍 1
alexm changed title from #6911 save on enter to WIP: #6911 save on enter 2024-05-15 06:06:57 +00:00
jsegarra commented 2024-05-16 10:03:23 +00:00
Member
Copy Link

Hay casos (sobre todo con QTextArea) en los que al hacer "enter" se quería un salto de linea:
Como en http://localhost:9000/#/claim/1/notes
image

En estos casos como se solucionaría?
Cambiamos para que el salto de linea sea con "Ctrl + Enter", más al estilo RocketChat?
@jgallego

En rocket, los saltos de linea son Shift(flecha)+Enter, a lo mejor es otra combinación de teclas a tener en cuenta

> Hay casos (sobre todo con QTextArea) en los que al hacer "enter" se quería un salto de linea: > Como en `http://localhost:9000/#/claim/1/notes` > ![image](/attachments/4a84957c-ed34-48f3-ac9e-6043065688e1) > > > En estos casos como se solucionaría? > Cambiamos para que el salto de linea sea con "Ctrl + Enter", más al estilo RocketChat? > @jgallego En rocket, los saltos de linea son Shift(flecha)+Enter, a lo mejor es otra combinación de teclas a tener en cuenta
jgallego commented 2024-05-16 12:55:22 +00:00
Owner
Copy Link

Hay casos (sobre todo con QTextArea) en los que al hacer "enter" se quería un salto de linea:
Como en http://localhost:9000/#/claim/1/notes
image

En estos casos como se solucionaría?
Cambiamos para que el salto de linea sea con "Ctrl + Enter", más al estilo RocketChat?
@jgallego

En rocket, los saltos de linea son Shift(flecha)+Enter, a lo mejor es otra combinación de teclas a tener en cuenta

El atajo "oficial" es shift+enter para hacer el soft enter

> > Hay casos (sobre todo con QTextArea) en los que al hacer "enter" se quería un salto de linea: > > Como en `http://localhost:9000/#/claim/1/notes` > > ![image](/attachments/4a84957c-ed34-48f3-ac9e-6043065688e1) > > > > > > En estos casos como se solucionaría? > > Cambiamos para que el salto de linea sea con "Ctrl + Enter", más al estilo RocketChat? > > @jgallego > > En rocket, los saltos de linea son Shift(flecha)+Enter, a lo mejor es otra combinación de teclas a tener en cuenta El atajo "oficial" es shift+enter para hacer el soft enter
jcasado added 4 commits 2024-05-22 10:07:58 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
4de9895b38 refs: #6911-saveOnEnter
jcasado dismissed jgallego’s review 2024-05-22 10:07:58 +00:00
Reason:

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

jcasado added 1 commit 2024-05-22 10:47:14 +00:00
jcasado changed title from WIP: #6911 save on enter to #6911 save on enter 2024-05-22 10:47:23 +00:00
jcasado added 1 commit 2024-05-23 07:31:09 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
103e3fe08a Merge branch 'dev' into 6911-saveOnEnter
alexm requested changes 2024-05-24 06:12:49 +00:00
Dismissed
src/components/EditPictureForm.vue Outdated
@ -209,1 +209,3 @@
<QForm @submit="onUploadAccept()" class="all-pointer-events">
<QForm
@submit="onUploadAccept()"
@keyup.enter="onUploadAccept()"
alexm commented 2024-05-24 06:11:09 +00:00
Member
Copy Link

Esto ya no hace falta

Esto ya no hace falta
jsegarra commented 2024-05-31 11:33:12 +00:00
Member
Copy Link

d8bf95008a

https://gitea.verdnatura.es/verdnatura/salix-front/commit/d8bf95008a20c791714ac6e3beea18d8aefa5132
src/components/ui/VnNotes.vue Outdated
@ -49,3 +49,3 @@
autogrow
autofocus
@keyup.ctrl.enter.stop="insert"
@keyup.enter.stop="insert"
alexm commented 2024-05-24 06:10:55 +00:00
Member
Copy Link

Con este cambio creo que haria falta añadir un
@keyup.shift.enter.(stop o prevent) y prevenir el insert sino se quedan sin poder saltar de linea

Con este cambio creo que haria falta añadir un @keyup.shift.enter.(stop o prevent) y prevenir el insert sino se quedan sin poder saltar de linea
jsegarra commented 2024-05-31 11:40:40 +00:00
Member
Copy Link

5139bba5a0

https://gitea.verdnatura.es/verdnatura/salix-front/commit/5139bba5a0fd9b47d035bb25fd1334298c700731
src/components/ui/VnNotes.vue Outdated
@ -56,1 +54,3 @@
:title="t('Save (ctrl + Enter)')"
<template #append>
<QBtn
:title="t('Save (Enter)')"
alexm commented 2024-05-24 06:10:04 +00:00
Member
Copy Link

Cambiar traduccion

Cambiar traduccion
jsegarra commented 2024-05-31 11:40:28 +00:00
Member
Copy Link

492e333d2f

https://gitea.verdnatura.es/verdnatura/salix-front/commit/492e333d2fdb4aaa1d903f5b1f21d78e6641889f
src/pages/Entry/Card/EntryBuysImport.vue Outdated
@ -175,3 +175,3 @@
auto-load
/>
<QForm>
<QForm @keyup.enter="onSubmit()">
alexm commented 2024-05-24 06:12:16 +00:00
Member
Copy Link

Esto tampoco hace falta no?
Seria poner lo del @submit?

Esto tampoco hace falta no? Seria poner lo del @submit?
jsegarra commented 2024-05-31 11:42:58 +00:00
Member
Copy Link

25d5887a57

https://gitea.verdnatura.es/verdnatura/salix-front/commit/25d5887a577e8b721a047eac7f67da8eef7633cc
src/pages/InvoiceIn/Card/InvoiceInDescriptor.vue Outdated
@ -395,14 +395,6 @@ const createInvoiceInCorrection = async () => {
>
<QItemSection>{{ t('components.smartCard.downloadFile') }}</QItemSection>
</QItem>
<QItem
alexm commented 2024-05-24 06:11:36 +00:00
Member
Copy Link

Esta parte pq se quita?

Esta parte pq se quita?
src/pages/InvoiceOut/InvoiceOutGlobalForm.vue Outdated
@ -64,6 +64,7 @@ onMounted(async () => {
<QForm
v-if="!initialDataLoading && optionsInitialData"
@submit="makeInvoice(formData, clientsToInvoice)"
@keyup.enter="makeInvoice(formData, clientsToInvoice)"
alexm commented 2024-05-24 06:11:52 +00:00
Member
Copy Link

Esto tampoco hace falta no?

Esto tampoco hace falta no?
jsegarra commented 2024-05-31 12:35:39 +00:00
Member
Copy Link

e6a1c31119

https://gitea.verdnatura.es/verdnatura/salix-front/commit/e6a1c31119c54dbd689202f871310a778af0b57a
src/pages/Order/Card/OrderCatalogItemDialog.vue Outdated
@ -34,3 +34,3 @@
<template>
<div class="container order-catalog-item q-pb-md">
<QForm @submit.prevent="addToOrder">
<QForm @submit.prevent="addToOrder" @keyup.enter="addToOrder">
alexm commented 2024-05-24 06:11:57 +00:00
Member
Copy Link

Esto tampoco hace falta no?

Esto tampoco hace falta no?
jsegarra commented 2024-05-31 12:40:24 +00:00
Member
Copy Link

0ad839dfc6

https://gitea.verdnatura.es/verdnatura/salix-front/commit/0ad839dfc655dee059a1c32da599ffc6aadb2115
jsegarra started working 2024-05-31 10:04:54 +00:00
jsegarra approved these changes 2024-05-31 12:40:45 +00:00
Dismissed
src/pages/Claim/Card/ClaimBasicData.vue Outdated
@ -149,1 +149,3 @@
type="number"
type="textarea"
/>
<QInput
jsegarra commented 2024-05-31 11:41:28 +00:00
Member
Copy Link

Duplicidad de campos. Posible merge mal hecho
d5dc9d974c/src/pages/Claim/Card/ClaimBasicData.vue

Duplicidad de campos. Posible merge mal hecho https://gitea.verdnatura.es/verdnatura/salix-front/src/commit/d5dc9d974c18d7d55e7baa2978e49c1c7a14b058/src/pages/Claim/Card/ClaimBasicData.vue
src/pages/InvoiceIn/Card/InvoiceInBasicData.vue Outdated
@ -399,3 +411,3 @@
</template>
</FormModel>
<QDialog ref="editDmsRef">
<QDialog ref="editDmsRef" @keyup.enter="upsert">
jsegarra commented 2024-05-31 11:56:59 +00:00
Member
Copy Link

59f9cf4111

https://gitea.verdnatura.es/verdnatura/salix-front/commit/59f9cf411186d3512b3af5f5f01690222f430f3c
jsegarra added 10 commits 2024-05-31 12:40:50 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
0ad839dfc6 src/pages/Order/Card/OrderCatalogItemDialog.vue
jsegarra dismissed jsegarra’s review 2024-05-31 12:40:50 +00:00
Reason:

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

jsegarra stopped working 2024-05-31 12:40:52 +00:00
2 hours 35 minutes
jsegarra requested review from alexm 2024-05-31 12:40:55 +00:00
alexm requested changes 2024-06-06 07:46:40 +00:00
Dismissed
src/components/ui/VnNotes.vue
@ -21,3 +21,3 @@
const newNote = ref('');
const vnPaginateRef = ref();
function handleKeyUp(event) {
alexm commented 2024-06-06 07:45:18 +00:00
Member
Copy Link
    if (event.key === 'Enter') {
        event.preventDefault();
        if (!event.shiftKey) insert();
    }
``` if (event.key === 'Enter') { event.preventDefault(); if (!event.shiftKey) insert(); } ```
👀 1
jsegarra marked this conversation as resolved
jsegarra added 1 commit 2024-06-06 07:50:27 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
26c7b09b8a perf: improve handleKeyUp method
alexm requested changes 2024-06-06 07:57:26 +00:00
Dismissed
alexm left a comment
Member
Copy Link

He probado 3 veces los e2e de agencyworkCenter y en esta rama me fallan el primero y el tercero
En dev no me fallan

He probado 3 veces los e2e de agencyworkCenter y en esta rama me fallan el primero y el tercero En dev no me fallan
jsegarra added 3 commits 2024-06-06 22:59:21 +00:00
jsegarra requested review from alexm 2024-06-06 22:59:44 +00:00
jsegarra commented 2024-06-06 23:01:12 +00:00
Member
Copy Link

He probado 3 veces los e2e de agencyworkCenter y en esta rama me fallan el primero y el tercero
En dev no me fallan

Toda la lógica y sentido del mundo
El objetivo de la tarea es que al hacer enter se ejecute el onSubmit
El test que haces referencia estaba usando el evento nativo de cypress ({enter}) en el método type, por eso fallaba, porque la hacer enter ya no hace falta hacer click sobre el botón

> He probado 3 veces los e2e de agencyworkCenter y en esta rama me fallan el primero y el tercero > En dev no me fallan > Toda la lógica y sentido del mundo El objetivo de la tarea es que al hacer enter se ejecute el onSubmit El test que haces referencia estaba usando el evento nativo de cypress ({enter}) en el método type, por eso fallaba, porque la hacer enter ya no hace falta hacer click sobre el botón
alexm approved these changes 2024-06-07 05:05:25 +00:00
jsegarra merged commit 53c47301c7 into dev 2024-06-07 07:20:48 +00:00
jsegarra deleted branch 6911-saveOnEnter 2024-06-07 07:20:48 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jsegarra
alexm
Labels
Clear labels   
bug

Something is not working

  
CR / Tests passed

CR / Tests passed

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
invalid

Something is wrong

  
Missing Tests

Missing Tests

  
wontfix

This won't be fixed

No Label
bug
CR / Tests passed
duplicate
enhancement
invalid
Missing Tests
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
5 Participants
Notifications
Total Time Spent: 2 hours 35 minutes
jsegarra
2 hours 35 minutes
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#207
No description provided.
Delete Branch "6911-saveOnEnter"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?