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

WorkerCalendar #270

Merged
alexm merged 37 commits from :WorkerCalendar into dev 2024-04-16 12:26:20 +00:00
Conversation 4 Commits 37 Files Changed 13 +40
1 changed files with 40 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 3d1e34a5b0 - Show all commits

40
src/pages/Worker/Card/WorkerDateLabel.vue Normal file
View File

@ -0,0 +1,40 @@
<script setup>
defineProps({
color: {
type: String,
default: null,
},
avatarClass: {
type: String,
default: null,
},
selected: {
type: Boolean,
default: undefined,
},
});
defineEmits(['update:selected']);
</script>
<template>
<QChip
class="chip"
:selected="selected"
:style="selected ? { backgroundColor: color } : null"
jsegarra marked this conversation as resolved Outdated
jsegarra commented 2024-04-09 09:33:10 +00:00
Outdated
Review
Copy Link

En vez de null poner { backgroundColor: 'black' }. Así está en Salix

En vez de null poner { backgroundColor: 'black' }. Así está en Salix
wbuezas commented 2024-04-09 12:24:40 +00:00
Outdated
Review
Copy Link

Color de background cambiado, también apliqué color de texto blancó ya si no en light mode no se veía el texto.

Y decidí renombrar el componente para que se entienda mejor que es.

Commit: fb4f35a378

Color de background cambiado, también apliqué color de texto blancó ya si no en `light mode` no se veía el texto. Y decidí renombrar el componente para que se entienda mejor que es. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/fb4f35a378545a089111eab4c86e313b8982762a
:text-color="selected ? 'white' : null"
@update:selected="$emit('update:selected', $event)"
>
<QAvatar
:color="color"
:class="avatarClass"
:style="{ backgroundColor: color }"
/>
<slot />
</QChip>
</template>
<style scoped lang="scss">
jsegarra marked this conversation as resolved Outdated
jsegarra commented 2024-04-09 09:38:23 +00:00
Outdated
Review
Copy Link

Mmm...ya hay una clase en app.scss:82 con la misma regla. Considerar de cambiar

Mmm...ya hay una clase en app.scss:82 con la misma regla. Considerar de cambiar
wbuezas commented 2024-04-09 12:25:26 +00:00
Outdated
Review
Copy Link

En el commit de arriba eliminé la clase chip, junto a los otros cambios.

Commit: fb4f35a378

En el commit de arriba eliminé la clase `chip`, junto a los otros cambios. Commit: https://gitea.verdnatura.es/verdnatura/salix-front/commit/fb4f35a378545a089111eab4c86e313b8982762a
.chip {
background-color: var(--vn-light-gray);
}
</style>