Styles cleanup #89
Styles cleanup #89
Conversation
| @@ -280,6 +292,14 @@ | |||
| {% set counter = ( counter | default(0) ) + 1 %} | |||
| <tr> | |||
| <td class="font-normal text-muted nowrap small-cell">{{ counter }}</td> | |||
| {# | |||
There was a problem hiding this comment.
Ici @al2b on a toute la balise img alors qu'on dit dans le tableau qu'on va afficher l'URL de l'image
There was a problem hiding this comment.
Ah oui mais on voulait plutôt la balise. On adapte le titre ?
There was a problem hiding this comment.
Sur les landmarks oui je me souviens mais pas sur les images si ? Quel est l'intérêt ?
En fait je suis partagée : remonter toute la balise ça permet d'être uniforme avec ce qu'on fait sur les landmarks.
Mais en même temps cette colonne sert à identifier l'endroit où on a une anomalie. Autant un <nav> peut-être difficile à différencier d'un autre sur une page (et du coup chercher à récupérer les classes et les id ça met toutes les chances de notre côté), autant une image c'est son src qui est important pour l'identifier et aller corriger. J'ai peur que la balise entière soit superflue.
There was a problem hiding this comment.
J'avoue que je penche pour pas s'encombrer de toute la balise parce que c'est plus lisible. Pour sortir rapidement une première version d'Accesseo je peux changer le titre du tableau et créer une issue pour plus tard ? Comme ça on ré-ouvre le sujet quand on sera prêtes ?
There was a problem hiding this comment.
Décision commune -> later
No description provided.