Skip to content

Corrige l'accessibilité des graphiques#287

Merged
bellangerq merged 5 commits intomainfrom
259-les-graphiques-ne-sont-pas-accessibles
Feb 8, 2023
Merged

Corrige l'accessibilité des graphiques#287
bellangerq merged 5 commits intomainfrom
259-les-graphiques-ne-sont-pas-accessibles

Conversation

@bellangerq
Copy link
Copy Markdown
Collaborator

@bellangerq bellangerq commented Feb 1, 2023

  • Mise à jour des couleurs en fonction du thème (clair ou sombre)
  • Ajoute une alternative sous forme de tableau

Closes #259.

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 1, 2023

Deploy Preview for confiture ready!

Name Link
🔨 Latest commit a7cf49e
🔍 Latest deploy log https://app.netlify.com/sites/confiture/deploys/63e38005e192d5000818eb55
😎 Deploy Preview https://deploy-preview-287--confiture.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@bellangerq bellangerq marked this pull request as ready for review February 2, 2023 10:50
@bellangerq bellangerq requested a review from hissalht February 2, 2023 10:50
@hissalht
Copy link
Copy Markdown
Collaborator

hissalht commented Feb 3, 2023

Les couleurs des graphiques ne sont pas mises à jour lorsqu'on change le thème de couleur avec le bouton "Paramètres d'affichage".

image

Ici, les couleurs du thème foncé sont affichées avec le thème clair

@benoitdequick
Copy link
Copy Markdown
Contributor

j'aimerais bien avoir les chiffres sans aller sur un autre onglet.
Pour le camembert, c'est facile d'ajouter le XX% dans la légende. Pour les barres, c'est plus compliqué :(

@bellangerq bellangerq force-pushed the 259-les-graphiques-ne-sont-pas-accessibles branch from 9f13e7a to a7cf49e Compare February 8, 2023 10:57
@bellangerq bellangerq merged commit afb4c01 into main Feb 8, 2023
@bellangerq bellangerq deleted the 259-les-graphiques-ne-sont-pas-accessibles branch February 8, 2023 13:11
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.

Les graphiques ne sont pas accessibles

3 participants