Dev 42 align rs

[FalHuor]

Quelques fixes css + ajout du lien discord sur la vue fédé

[hugoattal]

Je suis pas super fan de faire plusieurs trucs dans la même PR (Fix #42+ changer l'ordre des liens + ajout de la région). Normalement ça devrait être deux PR différentes, là c'est un peu dur à relire... Bon c'est pas très grave, mais comme ça tu le sais pour la prochaine fois 🙂

[hugoattal]

Alors, je vais pas te faire un retour sur le code, mais plutôt sur l'aspect général de la PR:

L'alignement du texte

J'ai vu que tu avais centré le texte sur les cartes.

• Il faudrait que tu précises dans la description de la PR qui tu as fait ces modifications, surtout si elles ne sont pas liés à l'issue lié
• Centrer le texte est assez mauvais pour l'UX : c'est à proscrire la plupart du temps, surtout pour texte qui peut être sur plusieurs lignes. Dans la majorité des cas, on gardera l'alignement à gauche pour que la lecture soit plus simple (les lignes commencent toujours au même endroit).

Du coup, je préfèrerai que tu gardes le texte aligné à gauche pour les cartes 😅

L'ordre des liens

Avant discord était à la fin, maintenant discord est au début.

En vrai, why not, mais ça aurait été bien de le préciser dans la PR, là j'ai dû comparer la diff, c'est un peu fastidieux...

L'alignement en base

J'ai l'impression que t'as un peu trop complexifié la solution et le résultat est pas forcément le mieux, vu qu'on perd l'alignement des noms des assos.

De ce que j'ai compris, t'as voulu aligner en bas la région, l'école et les réseaux sociaux. Je sais pas si tu as remarqué, mais le div qui wrap tout ça (dont la classe est .association-card) a un display flex. Tu peux voir l'alignement dans ton inspecteur.

En voyant ça, tu peux constater qu'il suffit que le nom de l'asso prenne tous l'espace pour que le reste soit aligné en bas. Il suffit donc d'ajouter un flex-grow: 1 sur le titre pour que ça marche :

---

Voilà voilà ! Il faudrait que tu modifies la PR pour régler ces problèmes 🙂 Si tu as des questions, n'hésite pas 😉

[FalHuor]

Ouai je repartirais surement sur des Branches différentes pour faire ça un peu mieux, Et en fait c'est surtout qu'au début, j'avais une branche avec juste des idées que j'ai continués sur autre chose petit à petit 😅

Et au final, je voulais juste déjà push un truc un peu rapide pour régler un peu l'espacement (surtout sur le bas des cartes) (Je sais que le centrage est moins bien d'un point de vue accessibilité et que du coup tu pers quelques points de référencement :c)

[FalHuor]

Code séparé dans des branches distinctes