Skip to content

feat(front): apply design mockups#217

Merged
QuentinMadura merged 16 commits intodevfrom
feat-160/apply-mockup-style
Apr 30, 2025
Merged

feat(front): apply design mockups#217
QuentinMadura merged 16 commits intodevfrom
feat-160/apply-mockup-style

Conversation

@QuentinMadura
Copy link
Contributor

@QuentinMadura QuentinMadura commented Apr 29, 2025

@QuentinMadura
Copy link
Contributor Author

closes #160

@QuentinMadura QuentinMadura changed the title update style feat 160 : apply design mockups Apr 29, 2025
@QuentinMadura QuentinMadura changed the title feat 160 : apply design mockups feat#160 : apply design mockups Apr 29, 2025
@QuentinMadura QuentinMadura force-pushed the feat-160/apply-mockup-style branch from 0ab4c25 to 3147825 Compare April 29, 2025 11:21
@ludovicdmt
Copy link
Member

ludovicdmt commented Apr 29, 2025

En mode sombre je n'ai aps de texte dans la popup : #216
image

Copy link
Contributor

@Marc-AntoineA Marc-AntoineA left a comment

Choose a reason for hiding this comment

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

Ci je compare avec le figma :

  • il n’y a pas de bordure verte sur le bloc de la légende sur le figma (je trouve par ailleurs cette bordure trop visible) ;
  • Adapter la couleur du texte sur les légendes en fonction du background : du blanc sur du vert foncé par ex.
  • les coordonnées gps ne sont pas en noir dans ton implémentation
  • le message en bas à droite
  • j’ai l’impression que la navbar est trop large

Et sinon, trop bien ! Surtout le sélecteur de calques <3

message.value = ""
}, 2000)
toast.add({
severity: "success",
Copy link
Contributor

Choose a reason for hiding this comment

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

Peut-être adapter ce style aux couleurs iarbre ? Le vert m’a l’air diffréent.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done !

@ludovicdmt
Copy link
Member

Un petit ajout au changelog pourrait être sympa aussi :)

@QuentinMadura
Copy link
Contributor Author

@ludovicdmt j'ai supprimé le mode sombre en bcd0e77 . Tu peux me confirmer que ça fonctionne aussi chez toi ?

@QuentinMadura
Copy link
Contributor Author

QuentinMadura commented Apr 30, 2025

  • il n’y a pas de bordure verte sur le bloc de la légende sur le figma (je trouve par ailleurs cette bordure trop visible) ;

Effectivement les bordures étaient légèrement trop large (1px vs 0.5 sur les maquettes). J'ai ajouté une classe utilitaire pour pouvoir réutiliser cette bordure en 0.5

  • Adapter la couleur du texte sur les légendes en fonction du background : du blanc sur du vert foncé par ex.

Done !

  • les coordonnées gps ne sont pas en noir dans ton implémentation

Done !

  • le message en bas à droite

C'est à dire ? J'ai changé les couleurs des toasts en "success"

  • j’ai l’impression que la navbar est trop large

C'est bien 110px sur les maquettes. À voir avec @Geoffreydorne si on veut diminuer. Pour l'instant je propose de laisser comme c'est.

@ludovicdmt
Copy link
Member

@ludovicdmt j'ai supprimé le mode sombre en bcd0e77 . Tu peux me confirmer que ça fonctionne aussi chez toi ?

Ouai ça marche niquel, merci !

@ludovicdmt
Copy link
Member

C'est dans la maquette mais je trouve ça chelou que le code source et OSM contributors n'ont pas la même taille et ne sont pas alignés non ?

@QuentinMadura
Copy link
Contributor Author

@ludovicdmt yes je suis d'accord. Je propose de mettre l'info sur le code source au niveau des attributions. Ça permet d'éviter de devoir jouer avec la position d'un bouton en absolute, qui est (très pénible) à gérer

@ludovicdmt
Copy link
Member

@QuentinMadura j'aime bien ce que tu as fait pour le code source. Pour les 3 personnes qui vont cliquer dessus, je pense que ça ne valait pas le coup de l'avoir aussi visible.

Par ailleurs, j'ai juste trouvé un petit bug de popup :
image

@Marc-AntoineA
Copy link
Contributor

Ok pour moi.

@QuentinMadura QuentinMadura changed the title feat#160 : apply design mockups feat(front): apply design mockups Apr 30, 2025
@QuentinMadura QuentinMadura merged commit 0f38edf into dev Apr 30, 2025
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ETQ utilisateur.ice je veux une animation claire pour le copier-coller de la popup de carte

3 participants