UX/UI: Mise en avant du bouton "Postuler pour ce candidat"#5096
UX/UI: Mise en avant du bouton "Postuler pour ce candidat"#5096hellodeloo merged 2 commits intomasterfrom
Conversation
hellodeloo
changed the title
hellodeloo
force-pushed
the
deloo/add-introjs-to-job-seeker-list
branch
6 times, most recently
from
4ccb4c7 to
c9eb2f7
Compare
hellodeloo
added
the
1-recette-jetable
|
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
francoisfreitag
left a comment
francoisfreitag
left a comment
There was a problem hiding this comment.
A-t-on une stratégie pour retirer ces éléments de présentation ?
@francoisfreitag Oui, on en a parlé ce matin. Y'a une carte (avec une date) pour retirer l'introjs qui arrive. |
|
Edit : Suite à la réponse au-dessus, c'est ok alors ! |
francoisfreitag
force-pushed
the
deloo/add-introjs-to-job-seeker-list
branch
from
8c64db2 to
ca35455
Compare
| let introJsDone = localStorage.getItem('introJsBtnPostuler') === 'done'; | ||
| if (introJsDone) return; |
There was a problem hiding this comment.
Pourquoi ne pas utiliser le cookie? Pour éviter de l’envoyer au serveur?
There was a problem hiding this comment.
À moins que je me trompe, localStorage n'envoie rien au serveur, tout comme les cookies
There was a problem hiding this comment.
Les cookies sont envoyés au serveur, pas le localStorage.
There was a problem hiding this comment.
Ah je croyais les deux. Sinon, why localStorage ? Question d'habitude, je dirais. Si tu préfères cookie, je peux passer en cookie
There was a problem hiding this comment.
Il me semble qu’une version précédente utilisait https://introjs.com/docs/tour/options#dontshowagaincookie. Comme la lib propose de stocker la préférence pour nous, j’aurais tendance à lui déléguer le problème 🤷
There was a problem hiding this comment.
Oui mais je ne peux pas utiliser leur système de cookie sans afficher la case à cocher "ne plus afficher" (ce qu'on ne veut pas). Du coup, j'ai refait en version cookie (durée de vie a 90 jours, car on doit retirer l'introjs a 90 jours, donc aide mémoire si jamais on oublie de la faire)
hellodeloo
force-pushed
the
deloo/add-introjs-to-job-seeker-list
branch
from
ca35455 to
adc9757
Compare
francoisfreitag
left a comment
francoisfreitag
left a comment
There was a problem hiding this comment.
Ça m’a l’air bien, j’ai poussé quelques améliorations JS. Tu peux les squash si elles te conviennent et merger.
| doneLabel: 'Fin', | ||
| steps: [{ | ||
| title: 'Postulez directement pour ce candidat !', | ||
| element: document.querySelector('#introJsBtnPostuler'), |
There was a problem hiding this comment.
Un peu plus clair et plus rapide.
| element: document.querySelector('#introJsBtnPostuler'), | |
| element: document.getElementById('introJsBtnPostuler'), |
hellodeloo
force-pushed
the
deloo/add-introjs-to-job-seeker-list
branch
from
412c4b6 to
b6e8420
Compare
3 participants
🤔 Pourquoi ?
Mettre en avant la nouvelle fonctionnalité pour "Postuler pour un candidat"