#99 Add vaccination eligibility government link #109
Conversation
|
Pour information, l'implémentation actuelle donnerait ça : J'aurais bien mis un peu plus en avant l'information, via un "panel" (bon, ça n'existe plus les panels dans bootstrap >3) avec un ℹ️ informatif Après, la question principale est de savoir si un lien sortant vers le site du gouvernement n'est pas ennuyeux d'un point de vu SEO (@francoisBouchet semblait assez catégorique sur le sujet hier..) |
Je viens de voir la conversation sur le Mattermost. Merci pour le signalement, j'avais fait un grand coup de molette sur la journée car trop occupé par le travail. 😇 |
|
Moi personnellement je serais plutôt très en faveur de ce lien qui est utile pour les utilisateurs. On a énormément de gens qui pensent que parce que le rdv apparaît, alors ils peuvent prendre le rdv les yeux fermés. |
|
Après pas mal de discussions sur le sujet sur Mattermost, je pense qu'il faudrait effectuer les changements suivants :
|
|
@fcamblor je viens de finir une contribution sur owasp dependency check. Je bosse sur tes remarques à ma prochaine disponibilité (demain soir je pense). |
|
Disparition après 10s.
|
src/views/vmd-rdv.view.ts
Outdated
| @@ -76,6 +78,14 @@ export abstract class AbstractVmdRdvView extends LitElement { | |||
| this.derniereCommuneSelectionnee = commune; | |||
| } | |||
|
|
|||
| protected firstUpdated(_changedProperties: PropertyValues) { | |||
| super.firstUpdated(_changedProperties); | |||
| this.eligibilityCriteriaTimeout = setTimeout(async () => { | |||
There was a problem hiding this comment.
je pense qu'il faudrait faire un clearTimeout dans le disconnectedCallback
src/views/vmd-rdv.view.ts
Outdated
| if (this.eligibilityCriteriaDisplayed) { | ||
| return html` | ||
| <div class="eligibility-criteria"> | ||
| <p>Soyons citoyens et respectons les conditions d'éligibilité :-)</p> |
There was a problem hiding this comment.
Contenu à peut-être changer en quelque chose de plus engageant ?
Quelque chose du type "
(c'est peut-être un peu long...)
There was a problem hiding this comment.
Je propose "Les critères d'éligibilité seront vérifiés pour une prise de rendez-vous" 72 char vs 62 pour "Soyons citoyens et respectons les conditions d'éligibilité :-)".
There was a problem hiding this comment.
"
(pour 3 caractères de plus je trouve que c'est mieux perso)
|
Pour info, j'ai créé une branche sur le repo qui est du coup visible ici : https://dev.vitemado.se/109-lien-eligibilite/ |
| @@ -76,6 +76,21 @@ | |||
|
|
|||
| } | |||
|
|
|||
| .eligibility-criteria { | |||
There was a problem hiding this comment.
Il faudrait peut-être rajouter un petit coup d'animation pour fadein/fadeout la div progressivement lors de l'affichage/masquage
There was a problem hiding this comment.
Je regarde ce point.
nhumblot
force-pushed
the
99-module-eligibilite
branch
3 times, most recently
from
0fbb592 to
d63225e
Compare
|
@fcamblor, finalement avec ta demande de fade-in, fade-out, je suis venu avec une solution complètement en css ce qui rend le code, je pense, plus simple. Test effectué sur Chrome et Firefox (Desktop Linux & Smartphone Android).
|
nhumblot
force-pushed
the
99-module-eligibilite
branch
from
d63225e to
fd16462
Compare
|
ah oui, trop bien de tout faire juste en CSS 👍 |
Fix #99