-
Notifications
You must be signed in to change notification settings - Fork 161
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Zep 04 : refonte de la page d'accueil #1833
Conversation
initialisation de la zep completude du design de la home
Est-ce que tu aurais les maquettes originales ? Elles ne sont plus disponibles sur la ZEP. |
@@ -368,6 +369,10 @@ | |||
'max_post_length': 1000000, | |||
'top_tag_max': 5, | |||
}, | |||
'social': { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tu as oublié de mettre les bonnes urls ici !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
étant donné qu'il s'agit du dictionnaire. Je pense que je vais plutot mêttre à jour le update.md pour garder le dictionnaire principal indépendant du site.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je vois pas pourquoi on enlèverait les liens vers les réseaux sociaux mais on garderait l'email de l'association. AMHA, soit on garde, soit on enlève mais on fait pas à moitié !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Si ça ne tenait qu'a moi on aurait viré tout ce qui est spécifique à zestedesavoir.com ici, mais ce n'est pas trop le scope de la ZEP/PR
Pour faire court, vu qu'on a attiré mon attention sur la PR. A noter : je ne reçoit pas de notification du repo, inutile de me ping ici, je ne recevrais rien, envoyez un MP au besoin.
Ces fugitif n'ont a priori rien à faire là, un oubli certainement Tout ce qui est rajouté dans https://github.com/zestedesavoir/zds-site/pull/1833/files#diff-f00f3aa7f238e444f350cbb1a7d0df38R1354 Cette propriété est à proscrire depuis 1753 (à peu près). A noter que c'est gulp (à moins que vous ayez mis un outils équivalent) qui gère les prefixes et autres choses du genre pour la compatibilité. https://github.com/zestedesavoir/zds-site/pull/1833/files#diff-e19c4aad376ee0e30990ef6c47c98149R2 Depuis quand un topic est un article ? Là ça donne la même sémantique à un article, un tuto et un sujet du forum. Or un sujet de forum c'est une suite de message, cf les metadata qui sont dans les templates. Et pour finir, le design ne respecte pas du tout le thème du site (et donc pas la maquette). La partie haute se voit moins large que le site (il faut retirer la marge à droite). Les liens ne sont pas alignés entre les colonnes, peut-être préférer un alignement en partant du bas, ou en jouant sur le line-height ou je ne sais trop quoi. La barre de recherche à gauche est alignée sur la fin du bloc de Clem (pile en dessous des 3 colonnes). La date des articles ne rend vraiment pas bien. Le titre a des marges trop grandes verticalement, pas assez à l'horizontale. La couleur du sous-titre n'est pas bonne. Il manque une bordure en dessous du sous-titre, à moins que ce soit souhaité, mais du coup on voit moins bien la fin du contenu vu que le fond du bloc et de la page sont confondus. La bordure autour des tags n'est pas souhaitable : le style du site veut que les bordures soient réservées aux éléments dont la couleur de fond est similaire à celle de son conteneur. Cf la maquette, ou alors refaire tout le style du site. La bulle des commentaires n'est pas assez grosse : sur la maquette elle peut accueillir de bien plus grandes valeurs. L'icône des sujets chauds ne respecte pas la convention présente partout sur le site : icônes monochromes. De plus, elle est laide, je trouve, et elle dépasse. Les sujets sont trop hauts, le but est de pouvoir en caser une dizaine, là ça n'est pas possible. L'interligne est trop important. Je tiens cependant à faire remarquer que malgré une non-doc le code ajouté par firm1 dans les SCSS est correctement placé et quasi-correctement formaté. Je tiens donc à te féliciter pour l'effort. |
En même temps tu as virer toutes les maquettes que tu avais faites (mise a part une qu'on a dut retrouver sur ton article de blog) et tu n'a pas non plus fournit les psd. Pourtant des demandes répétés on été faites pour avoir ces maquettes. De là je pense que tes remarques sont globalement pertinentes mais arrive un peu tard. Je pense que firm1 a fait ce qu'il a put avec ce qu'il avait. Pour moi ça peut fonctionner pour une v1 |
Un affichage différent est prévu pour les posts qui sont non lu ? |
Merci @Alex-D pour ton retour,
Tu devrais vraiment mettre à jour la doc sur ce que tu as fait, perso, je crois que pour cette ZEP, j'ai jamais autant fait de la retro-ingénierie.
Ah un oubli je suppose.
C'était vraiment moche sans la bordure, et je ne pense pas que ce soit très choquant. De toute façon la home sort un peu du cadre du design globale du site.
Tu avais visiblement trouvé une meilleure icone, mais vu que tu ne veux pas partager avec tout le monde, je fais avec ce que je trouve :( |
poulp a écrit
Ce n'était pas prévu dans la maquette, mais pourquoi pas, je ne sais pas encore comment faire ça. (background ou gras sur le texte) |
Est-ce que le design s'adapte aux mobiles ? |
@Situphen : c'est la moindre des choses qu'il soit responsive. |
Il y a un petit débordement de texte chez moi ! Je conseillerais d'agrandir un peu la marge entre le texte et les liens ("Découvrir les tutoriels" par exemple). Je verrai bien une petite transition pour adoucir le changement d'opacité au survol de l'image d'un tutoriel ou d'un article. (Avec |
@Situphen (et les autres) Si tu as des idées ou des fix tu peux les proposer en PR directement en partant de la branche zep-04 disponible sur upstream :) |
[ZEP 04] Petites améliorations, nettoyage et aération du code
word wrap + overflow hidden + ellipsis |
Pour info le screen a été MAJ avec les récents commits de @Situphen |
Est-ce que cette PR est encore nécessaire puisque Jiyong va refaire des nouvelles maquettes ? |
Aucune idée. Mais même si elle n'est pas parfaite, elle sera surement moins pire que ce qu'on a aujourd'hui. En attendant une maquette qui plait et convient à tout le monde, je serai d'avis de garder cette PR en cours, puisque son design semblait convenir à beaucoup de monde. On pourra toujours refaire une autre PR quand la nouvelle maquette sera bonne. |
Je suis d'accord pour dire que la home actuelle n'est pas parfaite mais est-ce judicieux de changer du tout au tout une home page à plusieurs reprises dans un laps de temps très court ? Je n'en suis pas convaincu. |
Perso je n'ai aucune idée de quand seront validées les prochaines maquettes. Donc le "laps de temps très court" peut durer longtemps. Y'a qu'a voir le temps ça a pris d'avoir ces maquettes |
Moi non plus mais si la home page est mise à jour pour la 1.4 avec le design actuel puis mise à jour une seconde fois dans un mois (ça reste un laps de temps très court pour moi) et qui change du tout au tout, ça va faire bizarre auprès des membres. |
Perso je trouve qu'en l'était c'est un WIP qui est loin de la qualité visuelle du reste du site (qui n'est déjà pas du grand art). Le rendu est vraiment brouillon. |
Et dire qu'à une époque tu étais fière de ton travail. C'est dommage :( |
J'ai une vision du site autre que celle de l'équipe. Pour moi, la ZEP-04 devait être rapide et mener vers une refonte partielle de l'ensemble du site, visuellement parlant. N'ayant plus à gérer la difficulté des templates, et ayant alors une API on aurait pu rendre le front-end fluide, asynchrone sur certaines choses, rajouter des transitions pour rendre le site plus "smooth". J'en suis techniquement relativement fiers. Au niveau du design, je suis bien conscient qu'il y a des tas de choses que l'on pourrait revoir. Je les applique désormais ailleurs toutes ces choses que je voyais applicables sur ZdS, puisque ça prenait du temps et qu'il fallait booster pour sortir le site. Quand j'aurais pu commencer à m'amuser on m'a demandé une doc. Tout le monde s'en sort sans désormais et le code que je vois jusqu'ici est loin d'être moche (sauf quand je le dis, mais je fournis les infos nécessaires à ce que ce soit joli, malgré mon détachement du dev de ZdS). En somme, je suis développeur front-end ayant un certain goût pour le webdesign sans pour autant être webdesigner. Il y a des gens bien plus qualifiés dans ce domaine, mais sont très prisés et donc toujours occupés. Un site comme ZdS n'est pas intéressant en terme de webdesign, donc comme pour le front-end ça n'attire pas les foules. |
Bon, on a besoin d'avis par ici. Etant donné que cette PR ne fais que des modifs de css, pour caler le design sur 3 colonnes avec 2-3 modifs par ci par là, je crois qu'on peux la faire passer, ça ne peux qu'améliorer les choses. Si une autre page d'accueil arrive après ce n'est pas très grave. ça ne deviendra grave que si la nouvelle page est moins bien que la précédente. Donc, j'aimerai donc que quelqu'un me confirme si ça vaut le coup de faire des quelques modifs de pollisage pour passer une QA dessus, ou on peut dores et déjà fermer la PR ? |
Je suis d'avis de fermer la PR pour toutes les raisons données précédemment et parce que les nouvelles maquettes s'écartent fortement de la solution que tu proposes là. A mon avis, la finalité de cette ZEP pourrait presque amener à une version 2 de Zeste de Savoir si nous répercutons le changement de design à toutes les pages. Mais ça, c'est encore autre chose. |
Bon comme je le disais au ZestMeeting, je vais faire une longue pause sur le dev et de toute façon cette PR n'a plu lieu d'être. Du coup je ferme cette PR pour ne pas polluer la zone PRs |
Cette PR a pour but d'introduire le design de la nouvelle page d'accueil. Les modifications visuelles effectuées sont :
Note pour QA
gulp build
) et testezScreens Shot.