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

fix: refs #7699 add icons and hint #1123

Merged
alexm merged 18 commits from 7699-refactroViewPassword into dev 2025-01-08 13:20:17 +00:00
Conversation 0 Commits 18 Files Changed 11 +3 -8
1 changed files with 3 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 3953721424 - Show all commits

11
src/components/common/VnInputPassword.vue
View File

@ -2,11 +2,8 @@
import VnInput from 'src/components/common/VnInput.vue';
import { ref } from 'vue';
const model = defineModel({ type: [Number, String] });
const $props = defineProps({
modelValue: {
type: [String, Number],
default: null,
},
toggleVisibility: {
type: Boolean,
default: false,
@ -14,18 +11,16 @@ const $props = defineProps({
});
const showPassword = ref(false);
const model = defineModel({ type: [Number, String] });
</script>
<template>
<VnInput
v-bind="{ ...$attrs }"
carlossa marked this conversation as resolved
jsegarra commented 2025-01-04 09:57:33 +00:00
Review
Copy Link

Si ya tenemos defineModel porque nos hace falta la props de modelValue?

Si ya tenemos defineModel porque nos hace falta la props de modelValue?
jsegarra commented 2025-01-04 09:58:58 +00:00
Review
Copy Link

Bueno, seria al reves 😅

Bueno, seria al reves 😅
v-model.number="model"
v-model="model"
:type="
$props.toggleVisibility ? (showPassword ? 'text' : 'password') : $attrs.type
"
hint=""
>
<template #append>
<template #append v-if="toggleVisibility">
<QIcon
:name="showPassword ? 'visibility_off' : 'visibility'"
class="cursor-pointer"
jsegarra commented 2025-01-03 16:20:09 +00:00
Review
Copy Link

Si lo quitamos, afectaría algo?

Si lo quitamos, afectaría algo?