Conversation
|
WORK IN PROGRESS TODO :
|
gcruchon
changed the title
| Un badge indique le statut d’un élément auquel il est rattaché. | ||
|
|
||
| <img src="../../assets/images/badge/badge-01.jpg" alt="badge" class="tk-markdown__img-fullscreen" /> | ||
| <p> |
There was a problem hiding this comment.
C'est une règle de MD-Lint qui dit qu'on ne doit pas mettre de inline HTML... mais je ne pense que c'est inutile, je vais les virer...
| - Univers AXA: rattaché à un univers métier d’AXA, comme assurance auto, maison etc. | ||
| - Un badge peut être utilisé dans des composants tels que des tableaux, des groupes de listes, un bloc de restitution. | ||
| - Les badges se déclinent en plusieurs couleurs, sémantiques ou contextuelles. Chaque couleur doit être utilisé en accord avec le sens rattaché : | ||
| - Vert : succès, |
There was a problem hiding this comment.
Mettre un espace pas le code d'un espace
There was a problem hiding this comment.
C'est la différence entre un espace et un espace insécable (non-breakable space). Devant un deux-points (:) il faut mettre un insécable, pas un espace "classique" (tu ne veux pas que le signe de ponctuation soit séparé du mot qui le précède ou le suit)
Idem pour les guillemets
| </p> | ||
|
|
||
| a) Position | ||
| ### a) Position |
There was a problem hiding this comment.
Le marqueur ### permet déjà de savoir que c'est un titre donc peut être supprimer le a)
There was a problem hiding this comment.
Si @samuel-gomez est OK, pas de souci pour moi. Je voulais le faire aussi... mais je ne l'ai pas fait pour ne pas dénaturer le texte.
There was a problem hiding this comment.
ok pour moi , globalement la hiérarchisation des contenus est à revoir
| a.tk-contributor__link(href="https://github.com/xballoy" title="xballoy") xballoy | ||
| img.tk-contributor__avatar(src="https://avatars2.githubusercontent.com/u/686305?v=4" loading="lazy") | ||
| span.tk-contributor__badge #2 | ||
| span.tk-contributor__commits Commits : 34 |
There was a problem hiding this comment.
Elle l'est :-D
generateContributors.js ligne 45.
Après, je suis d'accord que c'est mieux si c'est un .md qui est généré et pas un .pug. On ouvre une issue?
There was a problem hiding this comment.
ça reviendra au même, à la fin c'est de l'HTML :) , le markdown est utile quand on souhaite pouvoir en modifier le contenu manuellement, ce qui ne sera jamais le cas ici.
|
J'ai pushé des corrections sur la PR, reste encore à faire :
|
| indent_style = space | ||
| indent_size = 4 | ||
| trim_trailing_whitespace = false | ||
|
|
There was a problem hiding this comment.
ce n'est pas déjà fait dans ton autre PR ?
There was a problem hiding this comment.
Si, je savait pas quelle PR allait arriver avant :-D
| Pour aller plus loin: [https://www.w3.org/Translations/WCAG20-fr/](https://www.w3.org/Translations/WCAG20-fr/) | ||
|
|
||
| <img src="../../assets/images/accessibility/daltoniens.jpg" alt="daltoniens" class="tk-markdown__img-fullscreen" /> | ||
| <p><img src="../../assets/images/accessibility/daltoniens.jpg" alt="daltoniens" class="tk-markdown__img-fullscreen" /></p> |
There was a problem hiding this comment.
pourras-tu faire une repasse pour enlever tous les 'p' autour des images, stp ?
4 participants
Fix #5
I fixed: