Skip to content
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

feat(front): apply basic iarbre design #56

Merged
merged 11 commits into from
Feb 14, 2025

Conversation

Marc-AntoineA
Copy link
Contributor

@Marc-AntoineA Marc-AntoineA commented Feb 13, 2025

Specs :

  • Site vitrine: utilisation de variables scss pour les couleurs et les fonts ;
  • Front : ajout de la charte graphique de base : couleurs & fonts (mêmes noms de variables que pour le site vitrine)
  • Front : Mise en forme de la légende "comme le wireframe" avec l'affichage en "hexagones" notamment ;
  • Front: suppression du footer. Création d’une navbar hyper basique avec simplement le logo iarbre et la bonne couleur de fond
  • Upgrade des dépendances JS (pour éviter d'avoir plein de commits dus au dependency bot à la con)

Resolve #46

👇 Résultat
image

@ludovicdmt ludovicdmt self-requested a review February 14, 2025 09:16
Copy link
Member

@ludovicdmt ludovicdmt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Niquel pour moi c'est bien plus beau comme ça !

Comment on lines +45 to +90
.hexagon
&.scale-0
background-color: $scale-0
color: $brown
&::before
border-bottom-color: $scale-0
&::after
border-top-color: $scale-0
&.scale-2
background-color: $scale-2
color: $brown
&::before
border-bottom-color: $scale-2
&::after
border-top-color: $scale-2

&.scale-4
background-color: $scale-4
color: $brown
&::before
border-bottom-color: $scale-4
&::after
border-top-color: $scale-4

&.scale-6
color: $brown
background-color: $scale-6
&::before
border-bottom-color: $scale-6
&::after
border-top-color: $scale-6

&.scale-8
color: $brown
background-color: $scale-8
&::before
border-bottom-color: $scale-8
&::after
border-top-color: $scale-8

&.scale-10
background-color: $scale-10
&::before
border-bottom-color: $scale-10
&::after
border-top-color: $scale-10
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quel enfer de devoir définir ça à la main comme ça!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il y a probablement moyen de faire autrement mais cela dépassait ma compréhension sur le coup, d'autant plus qu'il n'y a "que 4 couleurs". Je vais quand même demander aux collègues s'iels avaient eu une autre option.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cette amélioration éventuelle sera traitée dans une autre MR.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah ok je ne pensais pas que c'était une possible de faire ça autrement. C'était pour moi un commentaire sur Vue en général et pas ton code en particulier !

@Marc-AntoineA Marc-AntoineA merged commit e5ce9f4 into dev Feb 14, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ETQ user, l'interface générale + légende de carte.iarbre.fr est globalement cohérente avec les wireframes
2 participants