Conversation
manuhabitela
left a comment
There was a problem hiding this comment.
Yo ! Merci pour tout ça, beau boulot :)
J'ai review uniquement la gaufre qui m'a pris un poil de temps déjà, si tu veux une autre paire d'yeux sur les autres widgets tu me redis, je pourrai regarder aussi. Ceci étant dit je suppose que les retours sur la gaufre peuvent t'inspirer sur les zautres widgets si besoin.
| @echo "" | ||
| @echo "$(GREEN)🎉 Bootstrap completed successfully!$(RESET)" | ||
| @echo "" | ||
| @echo "$(BOLD)Next steps:$(RESET)" |
There was a problem hiding this comment.
tous les "echo" formatés avec bold/couleurs etc ne sont pas formatés chez moi. Utiliser plutôt des @echo -e ... ?
There was a problem hiding this comment.
ah mince "ça marche chez moi" :/ quel shell tu utilises ?
manuhabitela
left a comment
There was a problem hiding this comment.
Merci pour tous les changements !
Ça m'a refait réfléchir un poil…
| }); | ||
| </script> | ||
|
|
||
| <button type="button" onclick="toggleGaufre(this)" aria-label="Ouvrir la Gaufre" aria-expanded="false"> |
There was a problem hiding this comment.
Ici non plus je ne parlerais pas de gaufre sur le label (voir com' plus haut)
There was a problem hiding this comment.
Et à relire, je mettrais pas la notion de "ouvrir/fermer" dans le label, vu que le aria-expanded l'exprime et est bien supporté.
| window._lasuite_widget = window._lasuite_widget || []; | ||
| _lasuite_widget.push(['lagaufre', 'init', { | ||
| api: 'https://lasuite.numerique.gouv.fr/api/services', | ||
| label: "La Gaufre", |
There was a problem hiding this comment.
j'sais pas si je loupe un truc ici mais j'ai l'impression que le label est pas passé correctement ? sur la page de démo j'ai toujours "Services" en label réellement rendu.
There was a problem hiding this comment.
J'ai pas pu reproduire, ou déjà fixé. Tu pourras retester ?
This simplifies the integrations and slightly widens the scope of the script. + Some a11y fixes.
| label: "Services de la Suite numérique", | ||
| closeLabel: "Fermer le menu", | ||
| headerLabel: "À propos", | ||
| loadingText: "Chargement…" |
There was a problem hiding this comment.
les … au lieu de ..., on aime 👌 👌
manuhabitela
left a comment
There was a problem hiding this comment.
Super tes modifs, merci à toi, beau boulot m'sieur !
https://lasuite-integration-pr33.osc-fr1.scalingo.io/guides/gaufre-v2/