Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A11y / issue #3482 - Suppression des type="info" apparaissant dans le HTML #3504

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

logic-fabric
Copy link
Collaborator

Certains composants, des <div> notamment, étaient rendues avec un attribut type="info"invalidant le HTML.

Cette PR renomme la prop type par $type, comme le recommande la doc de styled-components (je ne sais plus où mais je l'ai lu un jour 😛 ), dans les composants concernés :

  • <Message />
  • <Chip />
  • <HelpButtonWithPopover />

Lien vers l'issue : #3482

Copy link

netlify bot commented Mar 18, 2025

Deploy Preview for nostalgic-mahavira-52b682 canceled.

Name Link
🔨 Latest commit 8cbfceb
🔍 Latest deploy log https://app.netlify.com/sites/nostalgic-mahavira-52b682/deploys/67d98f85279f980007276dac

@logic-fabric logic-fabric changed the title A11y/isue 3482 - Suppression des type="info" apparaissant dans le HTML A11y / issue 3482 - Suppression des type="info" apparaissant dans le HTML Mar 18, 2025
@logic-fabric logic-fabric changed the title A11y / issue 3482 - Suppression des type="info" apparaissant dans le HTML A11y / issue #3482 - Suppression des type="info" apparaissant dans le HTML Mar 18, 2025
@github-actions github-actions bot temporarily deployed to 3504 March 18, 2025 15:26 Inactive
Copy link

@logic-fabric logic-fabric self-assigned this Mar 18, 2025
@logic-fabric logic-fabric requested a review from newick March 18, 2025 15:26
@newick newick added this to the Accessibilité 2025 milestone Mar 18, 2025
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.

2 participants