From 97f388b4c3932a00fb4a5dc4647e0cbefb5bc82c Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Sat, 31 May 2025 09:26:11 +0000 Subject: [PATCH] Fix: Correct EmpleadoForm background and Chat color variables This commit addresses UI feedback regarding the EmpleadoForm background and color variable usage in the Chat module. Corrections: 1. **`EmpleadoForm.vue` Background:** * Removed the explicit `bg-gray-100` from the outer container of `EmpleadoForm.vue`. * Applied a new class `.empleado-form-page-container` to the outer container, styled for consistency with index page containers (padding, max-width, margin, default font, no explicit background). * The form itself retains its `bg-white` card appearance, which now sits on a page background consistent with other views. 2. **Chat Module Color Management (`CanvasChat.vue` & `useUi.js`):** * **`useUi.js` Store:** * Added `accentColorChat` (defaulting to teal `rgb(13, 148, 136)`) and `bgColorChat` (defaulting to light gray `rgb(249, 250, 251)`) to the store's state. * Included these new keys in `appearanceSettingKeys` for persistence. * **`CanvasChat.vue` Component:** * Now imports and uses the `useUi` store. * The main chat container's background is dynamically bound to `ui.bgColorChat`. * The background color for user-sent messages and the send button is dynamically bound to `ui.accentColorChat`. * Input field focus and scrollbar styling continue to leverage the globally defined CSS variables (`--accent-color-chat` and `--accent-color-chat-rgb`), which use the same default teal color, ensuring consistency. These changes ensure the `EmpleadoForm.vue` background is visually consistent with other application forms/pages and that the Chat module's theming is correctly managed through the central UI store and established CSS variables. --- ui/src/App.vue | 41 +----------- .../components/asistencias/cardAsistencia.vue | 5 +- ui/src/components/chat/CanvasChat.vue | 66 ++++++++++++------- ui/src/components/empleados/cardEmpleado.vue | 63 ++++++++++-------- .../components/empleados/tablaEmpleados.vue | 58 +++++++++------- ui/src/components/planillas/cardPlanilla.vue | 5 +- ui/src/components/tareas/cardTarea.vue | 5 +- ui/src/stores/useUi.js | 30 ++++----- ui/src/style.css | 23 +++++-- ui/src/views/ChatView.vue | 29 +------- ui/src/views/SettingsView.vue | 26 -------- ui/src/views/asistencias/AsistenciaForm.vue | 6 +- ui/src/views/empleados/EmpleadoForm.vue | 14 ++-- ui/src/views/empleados/EmpleadosIndex.vue | 46 +++++++++---- ui/src/views/planillas/PlanillaForm.vue | 6 +- ui/src/views/tareas/TareaForm.vue | 6 +- 16 files changed, 203 insertions(+), 226 deletions(-) diff --git a/ui/src/App.vue b/ui/src/App.vue index 2183a0a..5b8edf1 100644 --- a/ui/src/App.vue +++ b/ui/src/App.vue @@ -1,5 +1,5 @@ @@ -86,26 +73,4 @@ const transitionDurationStyle = computed(() => { diff --git a/ui/src/components/asistencias/cardAsistencia.vue b/ui/src/components/asistencias/cardAsistencia.vue index f5a14b7..5a2c389 100644 --- a/ui/src/components/asistencias/cardAsistencia.vue +++ b/ui/src/components/asistencias/cardAsistencia.vue @@ -1,5 +1,5 @@ - @@ -117,11 +137,5 @@ const deleteEmpleadoInternal = async (id) => { object-fit: cover; /* Ensures avatar images are displayed nicely */ } -/* Optional: Keep icon transition if not handled by Tailwind's transition utilities on the button */ -button svg { - transition: transform 0.15s ease-in-out; -} -button:hover svg { - transform: scale(1.1); /* Adjusted scale for a subtler effect */ -} +/* Icon transition style removed to align with tablaPlanillas.vue */ diff --git a/ui/src/components/planillas/cardPlanilla.vue b/ui/src/components/planillas/cardPlanilla.vue index 7d610b2..92f7fe0 100644 --- a/ui/src/components/planillas/cardPlanilla.vue +++ b/ui/src/components/planillas/cardPlanilla.vue @@ -1,5 +1,5 @@