Skip to content
This repository has been archived by the owner on Jun 3, 2024. It is now read-only.

QMAPS-2801 ui fixes #1371

Merged
merged 4 commits into from
Jul 18, 2022
Merged

QMAPS-2801 ui fixes #1371

merged 4 commits into from
Jul 18, 2022

Conversation

xem
Copy link
Contributor

@xem xem commented Jul 12, 2022

Certaines des modifs faites dans cette PR surchargent des règles css présentes dans le design system. Certaines d'entre elles pourront être supprimées lorsque les problèmes de dark mode etc auront été résolus.

Appliquer la même shadow que le Date picker aux composants suivants :

  • notif sondage
  • popup reservation hotel (chambre/adulte/enfant)

Appliquer le font style serpPermaLink à la mention "Ouvert jusqu'à..." des établissements

Appliquer la couleur red-400-alpha20 en background au hover du CTA Favoris

Dans une fiche POI, appliquer la couleur green-500 à la note et aux étoiles seulement (le nombre d'avis reste en noir). Appliquer une font size 14 et un style bold à la note. Voir maquette https://zpl.io/z89gyMG

Arrondir les angles du border 1px grey-900-alpha16 dans le header vert

Notification de sondage:

  • Background color : grey-000  (/!\ nom de couleur chelou mais il s'agit bien de blanc
  • Rounded corners: 8px
  • Nouveau logo Qwant maps => j'ai demandé à Thomas ce qu'il voulait dire par là, l'image est customisée dans minerva et pas en dur.
  • CTA green 400 + Rounded corners 8px + green clair au hover
  • Icône "close" en 24x24px
  • Textes nom/description" en grey-900

@xem xem marked this pull request as draft July 12, 2022 14:54
@xem xem marked this pull request as ready for review July 13, 2022 14:04
@xem xem requested a review from pascaloliv July 13, 2022 14:04
@xem xem requested a review from pascaloliv July 13, 2022 14:52
@pascaloliv pascaloliv merged commit 988f620 into master Jul 18, 2022
@pascaloliv pascaloliv deleted the QMAPS-2801-fix-ui branch July 18, 2022 07:40
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants