#6891 fix: hide across only on disable #379

Merged
jorgep merged 1 commits from 6891-fixSearchbar into dev 2024-05-16 13:15:09 +00:00
Member
No description provided.
jorgep added 1 commit 2024-05-16 10:09:32 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
2855499148
fix: refs #6891 hide across only on disable
jorgep requested review from jsegarra 2024-05-16 10:10:53 +00:00
jorgep changed title from #6891 hide across only on disable to #6891 fix: hide across only on disable 2024-05-16 10:11:05 +00:00
jorgep reviewed 2024-05-16 10:11:36 +00:00
@ -83,4 +75,2 @@
<slot name="prepend" />
</template>
<template #append v-if="!!$attrs.disabled">
Author
Member

Así se ocultaba siempre.

Así se ocultaba siempre.
jorgep reviewed 2024-05-16 10:11:59 +00:00
@ -90,3 +82,2 @@
size="xs"
v-if="hover && value"
@click="handleValue(null)"
v-if="$slots.append && hover && value && !$attrs.disabled"
Author
Member

Poniendolo en el slot y el icono funciona. Aquí igual podríamos valorar crear una función. Lo que tu me digas.

Poniendolo en el slot y el icono funciona. Aquí igual podríamos valorar crear una función. Lo que tu me digas.
Member

Cuando limpias el input no te refresca la vista
Estoy en customer, y pongo dev, puslo enter y me saldrán 2 resultados
Si pulso la x, me elimina el input pero tengo que darle a enter para refrescar

Es correcto?

Cuando limpias el input no te refresca la vista Estoy en customer, y pongo dev, puslo enter y me saldrán 2 resultados Si pulso la x, me elimina el input pero tengo que darle a enter para refrescar Es correcto?
Author
Member

Cuando limpias el input no te refresca la vista
Estoy en customer, y pongo dev, puslo enter y me saldrán 2 resultados
Si pulso la x, me elimina el input pero tengo que darle a enter para refrescar

Es correcto?

Pues es el funcionamiento que había hasta ahora. A mí no me parece correcto, pero esto es solo para arreglar lo que había roto. Yo plantearía crear otra tarea para eso.

> Cuando limpias el input no te refresca la vista > Estoy en customer, y pongo dev, puslo enter y me saldrán 2 resultados > Si pulso la x, me elimina el input pero tengo que darle a enter para refrescar > > Es correcto? Pues es el funcionamiento que había hasta ahora. A mí no me parece correcto, pero esto es solo para arreglar lo que había roto. Yo plantearía crear otra tarea para eso.
jsegarra approved these changes 2024-05-16 13:12:28 +00:00
jorgep merged commit 475f0eda48 into dev 2024-05-16 13:15:09 +00:00
jorgep deleted branch 6891-fixSearchbar 2024-05-16 13:15:09 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 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#379
No description provided.