Skip to content

fix(dashboard): Masquer indicateur commentaire tuteur sur modules val…#56

Merged
Papoel merged 1 commit intomainfrom
frontend
Mar 27, 2026
Merged

fix(dashboard): Masquer indicateur commentaire tuteur sur modules val…#56
Papoel merged 1 commit intomainfrom
frontend

Conversation

@Papoel
Copy link
Copy Markdown
Owner

@Papoel Papoel commented Mar 27, 2026

🐛 FIX: Indicateur de commentaire incohérent sur modules validés

❗ Problème

Les modules déjà validés par le tuteur continuaient d’afficher :

  • un badge warning
  • un indicateur visuel (dot orange pulsant)

Cela introduisait une ambiguïté UX :
un module validé est censé signifier que les remarques ont été traitées,
mais l’interface suggérait encore une action en attente.


✅ Solution

  • Ajout de la condition suivante dans les filtres d’affichage : mentorValidatedAt is null
  • Restriction de l’affichage des indicateurs aux seuls modules :
  • contenant des commentaires
  • non encore validés par le tuteur

🎯 Comportement mis à jour

  • 🔶 Badge warning + dot pulsant :
  • affichés uniquement si commentaires non validés
  • 🟢 Modules validés :
  • conservent un fond bg-success-subtle
  • aucun indicateur parasite

🚀 Impact

  • Suppression du bruit visuel sur les modules validés
  • Meilleure lisibilité de l’état réel des modules
  • Guidage utilisateur plus précis vers les actions restantes
  • Cohérence métier renforcée entre validation et affichage

📁 Fichiers modifiés

  • templates/app/dashboard/_dashboardUserIndex.html.twig
  • lignes : 56, 88

📝 Remarques

Ce correctif s’inscrit dans l’amélioration globale de l’UX du dashboard :
les indicateurs deviennent désormais fiables et actionnables,
plutôt que simplement décoratifs.

…idés

- Ajout condition 'mentorValidatedAt is null' dans filtres Twig
- Badge warning et dot orange uniquement si module non validé
- Module validé = remarques prises en compte, plus besoin d'indicateur
@Papoel Papoel self-assigned this Mar 27, 2026
@Papoel Papoel merged commit 238dca0 into main Mar 27, 2026
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant