Skip to content

[A11Y] Retrait des <br> dans les crédits et les transcriptions#582

Merged
alexisben merged 6 commits intomainfrom
a11y-remove-br
Oct 21, 2024
Merged

[A11Y] Retrait des <br> dans les crédits et les transcriptions#582
alexisben merged 6 commits intomainfrom
a11y-remove-br

Conversation

@Olivia206
Copy link
Contributor

@Olivia206 Olivia206 commented Aug 21, 2024

Type

  • Nouvelle fonctionnalité
  • Bug
  • Ajustement
  • Rangement

Description

Suite côté thème de la PR #2149

Niveau d'incidence

  • Incidence faible 😌
  • Incidence moyenne 😲
  • Incidence forte 😱

Référence (ticket et/ou figma)

#570

Copy link
Contributor

@Clararigaud Clararigaud left a comment

Choose a reason for hiding this comment

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

All good, parcontre ca m'a permis de voir que dans les block cta, seul le premier < a > a un style button >

image

c'est une histoire de firstchild, je sais pas si tu veux en profiter pour fix ici ou si je fais une issue ?

@Olivia206
Copy link
Contributor Author

Olivia206 commented Aug 26, 2024

@Clararigaud c'est volontaire ça, seul le premier élément ressemble à un bouton, les autres à des liens

Si c'est bien de ça dont tu parles, le figma donne ça :
Capture d’écran 2024-08-26 à 10 45 58

@Clararigaud
Copy link
Contributor

Ah d'accord pardon j'avais pas vu lre figma, j'ai juste trouvé bizarre :)
Tout va bien alors !

@Olivia206
Copy link
Contributor Author

Haha oups faut le dire à Arnaud ça

@Olivia206
Copy link
Contributor Author

@Clararigaud @arnaudlevy j'attends que la PR des résumés soit prête pour merge ça ? Comme ça on envoie l'admin (qaui gère les 2) dans la foulée ?

@Olivia206 Olivia206 linked an issue Aug 28, 2024 that may be closed by this pull request
2 tasks
@alexisben
Copy link
Contributor

@SebouChu on peut valider cette osunyorg/admin#2149 ?

@alexisben alexisben merged commit a1e7cc9 into main Oct 21, 2024
@alexisben alexisben deleted the a11y-remove-br branch October 21, 2024 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a11y adjustment Incidence faible 😌 Risque faible de régression sur les sites standby

Projects

Status: Done
Status: Done

Development

Successfully merging this pull request may close these issues.

Suppression des sauts de ligne

3 participants