zyhou
commented
4 years ago Merci pour ta contribution :smiley:
Je trouve que c'est en effet une bonne idée de mettre un syteme de card sur X colonnes. Il y a tellement peu d'informations que mettre un talk par ligne n'est pas vraiment nécessaire. Mettre la date et le numéro d'edition (le #80) au dessus du titre est une bonne idée. J'aime beaucoup ta proposition de mettre le prochain talk sur une ligne, cela fait bien ressortir l'event. Bien joué le fait de gérer "prochainement" et "aujourd'hui" :+1:
Les gouts et les couleurs c'est toujours un sujet sensible, je trouve le rouge peut etre trop rouge pour la date/numéro d'édition. Tu en penses quoi? Je sais bien que pour l'instant nous avons pas vraiment d'identité visuelle...
Les border (box-shadow) sont un peu trop forte, j'ai l'impression que c'est un background au niveau de la liste. Entre les cartes on vois un petit cercle blanc. Peut etre essayer avec un rouge plus légé.
Si tu peux mettre un label sur ta PR (WIP pour dire en cours, et RFR pour dire PR qui peut etre merge), ou tu peux mettre une todolist dans la description de ta PR. N'hésite pas à me mettre en reviewer une fois la PR ready.
Une fois cette PR mergé, je peux t'ajouter en tant que contributeur si cela t'interesse, tu n'auras plus besoin de faire de fork pour proposer une PR.
Super travail en tout cas :clap: :clap:
aymericingargiola
allcontributors[bot]
resolve #46
Ajout du support de sass pour faire des choses custom/avancée en css et amélioration du design de la liste des talks