Skip to content

Conversation

slamer59
Copy link
Collaborator

@slamer59 slamer59 commented Sep 5, 2025

  • Correction des codes « Ville de Paris » et « Dijon Métropole » codés en dur dans MobileComparisonCard
  • Ajout des propriétés community1Name et community2Name au composant TransparencyComparison
  • Mise à jour du composant de page pour transmettre les valeurs community.nom à TransparencyComparison
  • La comparaison mobile affiche désormais les noms de communauté corrects de manière dynamique, comme sur la version bureau

Correction d'un problème d'affichage mobile où les noms de communauté affichaient toujours les valeurs par défaut

- Fixed hardcoded 'Ville de Paris' and 'Dijon Métropole' in MobileComparisonCard
- Added community1Name and community2Name props to TransparencyComparison component
- Updated page component to pass community.nom values to TransparencyComparison
- Mobile comparison now displays correct community names dynamically like desktop version

Fixes mobile display issue where community names were always showing default values
Copy link
Contributor

github-actions bot commented Sep 5, 2025

✅ Review app deployed

Check how your changes look like!

ℹ️ Name 🔗 Infos & links
🌱 Latest deployed commit 7e9a5e9
👁️ Preview https://13_eclaireur_public-PR-419.cleverapps.io

Copy link
Contributor

github-actions bot commented Sep 5, 2025

📸 Visual Regression Test Results

⚠️ Visual differences detected

📊 Test Artifacts

📋 Report Includes:

  • Visual comparisons (reference vs. test)
  • Highlighted diffs
  • Interactive viewer
    🕒 Artifacts are retained for 30 days.

Copy link
Collaborator

@oualter oualter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OKAY

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants