Affichage Footer #210
No reviewers
Labels
No labels
backend
bug
duplicate
enhancement
frontend
help wanted
invalid
mauvais-repo
Priorité: Basse
Priorité: Critique
Priorité: Haute
Priorité: Moyenne
question
refactor
suggestion
wontfix
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: agecem/agecem-org#210
Loading…
Reference in a new issue
No description provided.
Delete branch "fix/affichage-footer"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Ajout de footer.css
Ajout de balises div à index.html
@ -9,2 +2,2 @@
</footer>
{{ end }}
<footer class="pageFooter">
<div class="wrapper">
Pourquoi une classe qui s'appelle juste
wrapper
?Pour que le contenu du footer aie les mêmes contraintes que le contenu de la page, et permettre que le background du footer prenne toute la largeur.
@ -41,3 +41,2 @@
</div>
<!-- {{ template "snackbar-html" }} -->
{{ template "footer-html" }}
</div>
Pourquoi une nouvelle fermeture de div ici?
Parce que certains divs de la page index n'étaient pas fermés, ce qui fuckait avec l'affichage du footer.
Le footer est vraiment trop large, c'était sensé être quelque chose de discret dans le bas de la page mais ça prend beaucoup d'espace visuel (en mode mobile c'est encore pire)
also si ça pouvait faire comme sur https://git.agecem.com/ et que le footer apparaisse juste quand t'es scrolled all the way down ça serait vraiment mieux
J'ai produit le footer selon les pratiques courantes et en prenant en compte que le footer risque de contenir plus de stuff au fur et à mesure.
Si l'aspect discret était une priorité dans le design, soit c'était pas assez clair quand on s'en est jasé, soit je l'ai loupé. J'peux faire quelque chose de plus dans ce style-là si on veut de quoi de plus discret.
J'ai pas réduit la taille des fontes dans le footer mobile, faque ça je pourrai le faire aussi.
Pull request closed