[#ERP-42] Mettre à jour la lib Malio UI #16

Merged
tristan merged 6 commits from feature/ERP-42-mettre-a-jour-la-lib-malio-ui into develop 2026-05-28 12:27:33 +00:00
8 changed files with 46 additions and 47 deletions
Showing only changes of commit 0a5ac61957 - Show all commits
+3 -2
View File
@@ -4,6 +4,7 @@
<MalioSidebar
v-model="ui.sidebarCollapsed"
:sections="translatedSections"
:sidebar-class="ui.sidebarCollapsed ? '' : 'w-[232px]'"
>
<template #logo>
<img src="/LOGO_MALIO.png" alt="Malio"/>
@@ -16,10 +17,10 @@
<div class="h-full flex-1 flex flex-col min-h-0 min-w-0">
<SiteSelector v-if="showSiteSelector"/>
<main
class="flex flex-1 flex-col overflow-y-auto overflow-x-hidden bg-white px-4 pb-24 sm:px-8 lg:px-16">
class="flex flex-1 flex-col overflow-y-auto overflow-x-hidden bg-white px-4 pb-10 sm:px-6 lg:px-12 xl:px-[170px]">
Outdated
Review

🟡 Valeurs en dur dans le layout

  • sidebar-class="w-[232px]" ligne 9
  • xl:px-[170px] ici
  • h-[47px] ligne 22 pour la sticky bar (au lieu de l'ancien h-8 sm:h-12)

Pourquoi 232 / 170 / 47 ? Si ces valeurs viennent d'une maquette Figma, OK mais un commentaire à côté serait précieux. Sinon, des tokens Tailwind standards (w-58, lg:px-12, h-12) seraient préférables — ces arbitrary values rendent le layout fragile au moindre design tweak.

🟡 **Valeurs en dur dans le layout** - `sidebar-class="w-[232px]"` ligne 9 - `xl:px-[170px]` ici - `h-[47px]` ligne 22 pour la sticky bar (au lieu de l'ancien `h-8 sm:h-12`) Pourquoi 232 / 170 / 47 ? Si ces valeurs viennent d'une maquette Figma, OK mais un commentaire à côté serait précieux. Sinon, des tokens Tailwind standards (`w-58`, `lg:px-12`, `h-12`) seraient préférables — ces *arbitrary values* rendent le layout fragile au moindre design tweak.
Outdated
Review

J'ai ajouté des commentaire puisque c'est des valeurs venant d'une maquette.

J'ai ajouté des commentaire puisque c'est des valeurs venant d'une maquette.
<div
aria-hidden="true"
class="pointer-events-none sticky top-0 z-30 h-8 flex-shrink-0 bg-white sm:h-12"/>
class="pointer-events-none sticky top-0 z-30 h-[47px] flex-shrink-0 bg-white"/>
<slot/>
</main>
</div>
@@ -1,7 +1,7 @@
<template>
<div>
<h1 class="text-xl font-bold text-primary-500 sm:text-2xl">{{ $t('commercial.title') }}</h1>
<p class="mt-4 text-neutral-500">{{ $t('commercial.welcome') }}</p>
<PageHeader>{{ $t('commercial.title') }}</PageHeader>
<p class="text-neutral-500">{{ $t('commercial.welcome') }}</p>
</div>
</template>
@@ -1,13 +1,9 @@
<template>
<div>
<div class="flex items-center justify-between">
<h1 class="text-xl font-bold text-primary-500 sm:text-2xl">
{{ t('admin.auditLog.title') }}
</h1>
</div>
<PageHeader>{{ t('admin.auditLog.title') }}</PageHeader>
<!-- Filtres -->
<section class="mt-4 rounded border border-gray-200 bg-white p-4">
<section class="rounded border border-gray-200 bg-white p-4">
<!-- Labels uniformes au-dessus : les composants Malio sont utilises sans
leur `label` flottant interne pour ne pas mixer deux patterns de label.
A revoir une fois le composant calendar Malio développé -->
2
+12 -14
View File
@@ -1,22 +1,20 @@
<template>
<div>
<!-- En-tete -->
<div class="flex items-center justify-between">
<h1 class="text-xl font-bold text-primary-500 sm:text-2xl">
{{ t('admin.roles.title') }}
</h1>
<MalioButton
v-if="can('core.roles.manage')"
:label="t('admin.roles.newRole')"
icon-name="mdi:add-bold"
icon-position="left"
@click="openCreateDrawer"
/>
</div>
<PageHeader>
{{ t('admin.roles.title') }}
<template #actions>
<MalioButton
v-if="can('core.roles.manage')"
:label="t('admin.roles.newRole')"
icon-name="mdi:add-bold"
icon-position="left"
@click="openCreateDrawer"
/>
</template>
</PageHeader>
<!-- Table des roles -->
<MalioDataTable
class="mt-6"
:columns="columns"
:items="roleItems"
:total-items="roles.length"
+1 -7
View File
@@ -1,15 +1,9 @@
<template>
<div>
<!-- En-tete -->
<div class="flex items-center justify-between">
<h1 class="text-xl font-bold text-primary-500 sm:text-2xl">
{{ t('admin.users.title') }}
</h1>
</div>
<PageHeader>{{ t('admin.users.title') }}</PageHeader>
<!-- Table des utilisateurs -->
<MalioDataTable
class="mt-6"
:columns="columns"
:items="userItems"
:total-items="users.length"
+2 -2
View File
@@ -1,7 +1,7 @@
<template>
<div>
<h1 class="text-xl font-bold text-primary-500 sm:text-2xl">{{ $t('dashboard.title') }}</h1>
<p class="mt-4 text-neutral-500">{{ $t('dashboard.welcome') }}</p>
<PageHeader>{{ $t('dashboard.title') }}</PageHeader>
<p class="text-neutral-500">{{ $t('dashboard.welcome') }}</p>
</div>
</template>
+12 -14
View File
@@ -1,22 +1,20 @@
<template>
<div>
<!-- En-tete -->
<div class="flex items-center justify-between">
<h1 class="text-xl font-bold text-primary-500 sm:text-2xl">
{{ t('admin.sites.title') }}
</h1>
<MalioButton
v-if="can('sites.manage')"
:label="t('admin.sites.newSite')"
icon-name="mdi:add-bold"
icon-position="left"
@click="openCreateDrawer"
/>
</div>
<PageHeader>
{{ t('admin.sites.title') }}
<template #actions>
<MalioButton
v-if="can('sites.manage')"
:label="t('admin.sites.newSite')"
icon-name="mdi:add-bold"
icon-position="left"
@click="openCreateDrawer"
/>
</template>
</PageHeader>
<!-- Table des sites -->
<MalioDataTable
class="mt-6"
:columns="columns"
:items="siteItems"
:total-items="sites.length"
@@ -0,0 +1,12 @@
<template>
<!-- Entete de page standard : source unique du style des titres.
Slot par defaut = texte du titre, slot #actions = boutons a droite. -->
<div class="mb-[44px] flex items-center justify-between gap-4">
<h1 class="text-[32px] font-semibold text-primary-500">
<slot/>
</h1>
<div v-if="$slots.actions" class="shrink-0">
<slot name="actions"/>
</div>
</div>
</template>