search

ToulouseDataViz / website

L'espace de dévelopment du site web toulouse-dataviz.fr
2 stars 2 forks source link

Split hackaviz footer #108

Closed vvivanloc closed 3 years ago

vvivanloc commented 3 years ago

Suite à ta remarque, il serait intéressant de couper le footer en deux :

emaulandi commented 3 years ago

En tout cas il ne faut pas que le composant HackavizResults contienne le HackavizFooter. Il est important de découpler les rendus pour qu'on puisse être 100% modulaires, chaque composant doit pouvoir être utilisé dans un autre contexte.

emaulandi commented 3 years ago

Ce qui pourrait être intéressant, ce serait d'avoir un lien vers Edition précédente et A propos en haut et en bas des pages liées à l'hackaviz + les sponsors uniquement en bas pour simplifier la navigation

du type (style à définir)

image

vvivanloc commented 3 years ago

En tout cas il ne faut pas que le composant HackavizResults contienne le HackavizFooter. Il est important de découpler les rendus pour qu'on puisse être 100% modulaires, chaque composant doit pouvoir être utilisé dans un autre contexte.

Euh... le composant HackavizResults utilise le HackavizFooter. Ce dernier est réutilisé ailleurs.

emaulandi commented 3 years ago

Oui justement il ne faudrait pas. Si on veut mettre des résultat ailleurs sans footer ce n'est pas possible. Donc il fuadrait les décorréler. Si j'ai besoin dans une base d'avoir les résultat puis le footer, je met 2 composant HackavizResults et HackavizFooter, sinon je met juste HackavizResult

emaulandi commented 3 years ago

Par exemple, on utilise HackavizResult dans le composant Hackaviz pour montrer uniquement les gagnants, on aurait pu vouloir utiliser HackavizResults ou autre sans avoir besoin du footer

vvivanloc commented 3 years ago

fixed #118

vvivanloc commented 3 years ago

Retour d'Alain O : c'est déjà trop compliqué - rajouter les liens du footer en haut de page ajoute à la confusion.