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

#7124 - First Input Autofocus without property #251

Merged
jsegarra merged 5 commits from 7124_autofocus_first_input into dev 2024-03-25 07:17:04 +00:00
Conversation 0 Commits 5 Files Changed 5 +2 -15
1 changed files with 2 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit c4f7b5e7c0 - Show all commits

17
src/boot/qformMixin.js
View File

@ -1,11 +1,6 @@
import { QForm } from 'quasar';
import { getCurrentInstance } from 'vue';
export default {
inject: { QForm },
component: { QForm },
components: { QForm },
extends: { QForm },
mounted: function () {
const vm = getCurrentInstance();
if (vm.type.name === 'QForm')
@ -14,19 +9,11 @@ export default {
// AUTOFOCUS
jsegarra marked this conversation as resolved Outdated
alexm commented 2024-03-22 06:54:03 +00:00
Outdated
Review
Copy Link

Justo he probado en claimBasicData (ejemplo: http://localhost:9000/#/claim/1/basic-data) y es curioso pq no funciona. Dado que el primer elemento si es un input pero esta disabled. Igual se podría añadir que comprobase que no tenga la clase .disabled

Justo he probado en claimBasicData (ejemplo: http://localhost:9000/#/claim/1/basic-data) y es curioso pq no funciona. Dado que el primer elemento si es un input pero esta disabled. Igual se podría añadir que comprobase que no tenga la clase `.disabled`
jsegarra commented 2024-03-22 06:57:32 +00:00
Outdated
Review
Copy Link

Si, es una posibilidad que había visto pero no contemplado.
Lo tengo en cuenta para mas cambios

Si, es una posibilidad que había visto pero no contemplado. Lo tengo en cuenta para mas cambios
jgallego commented 2024-03-22 07:08:05 +00:00
Outdated
Review
Copy Link

me parece muy interesante que obvie los disables

me parece muy interesante que obvie los disables
🚀 1
const elementsArray = Array.from(this.$el.elements);
const index = elementsArray.findIndex(element => element.classList.contains('q-field__native'));
const firstInputElement = elementsArray.find(element => element.classList.contains('q-field__native'));
jsegarra marked this conversation as resolved
alexm commented 2024-03-22 06:05:06 +00:00
Review
Copy Link

No usar una variable si luego solo se usa una vez. Igual se te ha quedado de antes

No usar una variable si luego solo se usa una vez. Igual se te ha quedado de antes
😆 1
jsegarra commented 2024-03-22 06:34:19 +00:00
Review
Copy Link

Si que se usaba en el evento key, pero eso pertenece a otra rama

Si que se usaba en el evento key, pero eso pertenece a otra rama
if (index !== -1) {
const firstInputElement = elementsArray[index];
if (firstInputElement) {
firstInputElement.focus();
}
// KEYUP Event
document.addEventListener('keyup', function (evt) {
if (evt.keyCode === 13) {
that.onSubmit();
}
});
}
jsegarra marked this conversation as resolved
jgallego commented 2024-03-21 14:24:41 +00:00
Review
Copy Link

refactor de chatGpt para no usar -1
const firstInputElement = elementsArray.find(element => element.classList.contains('q-field__native'));

if (firstInputElement) {

refactor de chatGpt para no usar -1 const firstInputElement = elementsArray.find(element => element.classList.contains('q-field__native')); if (firstInputElement) {
👀 1
},
};