fix: refs #7699 add icons and hint #1123
No reviewers
Labels
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#1123
Loading…
Reference in New Issue
No description provided.
Delete Branch "7699-refactroViewPassword"
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?
WIP: fix: refs #7699 add icons and hintto fix: refs #7699 add icons and hintUff 3 campos iguales que solo cambian el label? y el model?
Veo que haria falta un VnInputPassword
@ -42,9 +42,14 @@ const $props = defineProps({
type: Number,
default: null,
},
toggleVisibility: {
Se podria hacer con $attrs en vez de añadir prop, no?
@ -125,3 +130,3 @@
v-model="value"
v-bind="{ ...$attrs, ...styleAttrs }"
:type="$attrs.type"
:type="toggleVisibility ? (showPassword ? 'text' : 'password') : $attrs.type"
Estamos añadiendo una condición a VnInput y debería estar en un componente aparte
@ -138,3 +143,4 @@
<slot name="prepend" />
</template>
<template #append>
<QIcon
Añdimos un icono condicional a VnInput.
Propuesta, crear VnInputPassword
@ -166,18 +167,3 @@ const handleInsertMode = (e) => {
</QInput>
</div>
</template>
<i18n>
ojo, que creo que nos hemos llevado cosas por delante, me refiero que hemos quitado código de dev, es intencional?
@ -135,3 +136,3 @@
:data-cy="$attrs.dataCy ?? $attrs.label + '_input'"
>
<template v-if="$slots.prepend" #prepend>
<template #prepend>
ojo, que creo que nos hemos llevado cosas por delante, me refiero que hemos quitado código de dev, es intencional?
Sí
@ -156,3 +157,3 @@
}
"
></QIcon>
/>
✅
@ -0,0 +7,4 @@
type: [String, Number],
default: null,
},
toggleVisibility: {
Si eliminamos la prop.modelValue, esto podría ser un attr, no?
Quedaría mas limpio, no?
@ -0,0 +14,4 @@
});
const showPassword = ref(false);
const model = defineModel({ type: [Number, String] });
Si ya tenemos defineModel porque nos hace falta la props de modelValue?
Bueno, seria al reves 😅
@ -0,0 +23,4 @@
:type="
$props.toggleVisibility ? (showPassword ? 'text' : 'password') : $attrs.type
"
hint=""
Si lo quitamos, afectaría algo?
@ -82,7 +81,7 @@ async function onSubmit() {
<template #prepend>
Es correcto que esté este icono?
Sí
@ -19,6 +19,7 @@ describe('Login', () => {
it('should fail to log in using wrong password', () => {
cy.get('input[aria-label="Username"]').type('employee');
cy.get('input[aria-label="Password"]').type('wrongPassword');
cy.get('.q-field__append > .q-icon');
Esta linea es para que el test no falle o que aporta? Quiero decir, sobre el elemento no se hace ninguna accion no?
Comprobar que esta por lo tanto lo pulsa
New commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings