search

CovidTrackerFr / vitemadose-front

Interface utilisateur de l'outil ViteMaDose
https://vitemadose.covidtracker.fr
Other
80 stars 51 forks source link

[Chronodose] Mise en place des highlight de centres #155

Closed fcamblor closed 3 years ago

fcamblor commented 3 years ago

Les cards de rdv doivent pouvoir être "highlightées" dans le cas où le centre fournit des rdv à court terme (dans les 48h)

Ça donne ça :

Vaccination_COVID-19_à_Villenave-d_Ornon_33140

Branche visible ici : https://dev.vitemado.se/tomorrow_appointments/ _(en attendant que le back nous mette à dispo le noeud appointment_schedules, les centres sont highlight dès lors qu'ils ont un nombre de rdv pair)_

bilelz commented 3 years ago

Hello, 👍 pour la feature. J'ai une première remarque pour le contraste du highlight qui n'est pas assez marqué https://webaim.org/resources/contrastchecker/?fcolor=5561D9&bcolor=DDDFF7

fcamblor commented 3 years ago

j'ai rebase/force push dans le prolongement de la PR #156 afin d'avoir une vue de ce qui serait "MEP-able" ce soir

fcamblor commented 3 years ago

Un truc avant que j'oublie : il faut faire attention car à priori, les highlights de créneaux ne devraient se faire sur les appointment_schedules dont le type de vaccin fait uniquement partie des ARNm (Pfizer/Moderna) d'après jjbaudet :-)

fcamblor commented 3 years ago

Je merge car il faut MEP au plus tôt ce matin (vu avec @rozierguillaume )

@bilelz je n'oublie pas ton commentaire, je pense qu'on itèrera sur les contrastes dans le courant de la journée si je trouve un peu de temps.

(et @Floby : j'attendrai d'intégrer la branche de refacto date-fns pour dégager DateUtils + j'ai vu avec @rozierguillaume qu'on merge mon rebase de #148 quand on aura un peu plus de recul sur les changements importants que tu as fait, c'est plus safe)