Date : 30/12/2025 Status : EN ATTENTE VALIDATION Priorité : Haute
Audit complet de l'admin QwikPress. Trois catégories de problèmes identifiées :
Règle Qwik : Pas de try/catch dans routeLoader$, server$, routeAction$. Qwik gère les erreurs.
| Fichier | Ligne | Action |
|---|---|---|
(pages)/pages/index.tsx |
154, 192 | Supprimer try/catch dans handlers |
(pages)/page/[pageId]/modules/add/index.tsx |
18 | Supprimer |
(pages)/page/[pageId]/settings/index.tsx |
174 | Supprimer |
(pages)/page/[pageId]/seo/index.tsx |
17 | Supprimer |
modules/[moduleType]/development/index.tsx |
40 | Supprimer |
modules/[moduleType]/layout.tsx |
29 | Supprimer |
modules/new/index.tsx |
12 | Supprimer |
modules/usage/index.tsx |
20 | Supprimer |
modules/common/new/index.tsx |
16 | Supprimer |
modules/common/[moduleId]/edit/fields/.../add/index.tsx |
21 | Supprimer |
modules/common/[moduleId]/edit/fields/.../edit/index.tsx |
21, 241 | Supprimer |
modules/common/[moduleId]/edit/index.tsx |
20 | Supprimer |
modules/common/index.tsx |
13 | Supprimer |
modules/registry/index.tsx |
11 | Supprimer |
modules/index.tsx |
25 | Supprimer |
config/footer/link-groups/[groupId]/index.tsx |
137 | Supprimer |
config/navigation/[navigationItemId]/edit/index.tsx |
109 | Supprimer |
config/navigation/index.tsx |
97 | Supprimer |
config/connectors/new/index.tsx |
68 | Supprimer |
config/redirections/new/index.tsx |
79 | Supprimer |
config/redirections/[id]/edit/index.tsx |
95 | Supprimer |
config/redirections/logs/index.tsx |
21 | Supprimer |
3 conventions mélangées :
admin-button / admin-button--primary (ancien)button / button--danger (très ancien)admin-btn / admin-btn--primary (composants UI)Tout harmoniser sur admin-btn (convention composants UI).
| Ancien | Nouveau |
|---|---|
admin-button |
admin-btn |
admin-button--primary |
admin-btn--primary |
admin-button--secondary |
admin-btn--secondary |
admin-button--outline |
admin-btn--ghost |
admin-button--small |
admin-btn--sm |
button--danger |
admin-btn--danger |
button--secondary |
admin-btn--secondary |
| Fichier | Classes à remplacer |
|---|---|
(pages)/page/[pageId]/layout.tsx |
admin-button, button |
(pages)/page/[pageId]/settings/index.tsx |
admin-button, button--danger |
(pages)/page/[pageId]/seo/index.tsx |
admin-button |
| Autres fichiers config | admin-button, button |
Styles inline avec couleurs en dur au lieu des variables CSS.
| Couleur hardcodée | Variable CSS |
|---|---|
#dc2626 |
var(--accent-danger) |
#6b7280 |
var(--text-tertiary) |
#374151 |
var(--text-secondary) |
#e5e7eb |
var(--surface-border) |
#6366f1 |
var(--accent-primary) |
(pages)/page/[pageId]/layout.tsx(pages)/page/[pageId]/settings/index.tsx(pages)/page/[pageId]/seo/index.tsxadmin/index.tsx (Dashboard)(pages)/pages/index.tsxlayouts/index.tsxcatchers/index.tsxhot-modules/index.tsx| Page | Problème | Correction |
|---|---|---|
page/[pageId]/settings/index.tsx |
<h2> direct |
Pas de header ici (layout parent) |
page/[pageId]/seo/index.tsx |
<h2> direct |
Pas de header ici (layout parent) |
page/[pageId]/content/index.tsx |
class="" vide |
Ajouter classe cohérente |
Les pages sous page/[pageId]/ ont un layout parent avec le header. Elles doivent seulement avoir un titre de section <h3> ou rien.
admin-button → admin-btnbutton-- → admin-btn--<h2> redondants dans pages enfantsJe propose de traiter fichier par fichier dans cet ordre :
(pages)/page/[pageId]/layout.tsx - Layout critique(pages)/page/[pageId]/settings/index.tsx - Page settings(pages)/page/[pageId]/seo/index.tsx - Page SEO(pages)/page/[pageId]/content/index.tsx - Page content(pages)/pages/index.tsx - Liste pagesPour valider : "OK on y va" ou "modifie [point]"