-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(nav): search by subtheme #168
Conversation
✅ Deploy Preview for ecospheres ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Quelques differences avec la maquette : Je n'ai pas mis les images et les tags dans la liste de tuiles :
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rebase pour qu'on puisse faire une review avec les changements qui ne concernent que cette PR.
9c5b78c
to
dae6912
Compare
dae6912
to
ee6d492
Compare
Manip git faite. La relecture est possible clairement maintenant |
ee6d492
to
6a7af23
Compare
2b53016
to
9858938
Compare
|
Oh je ne l'avais pas vu je vais le changer alors |
Un point pas bloquant (vu en daily) sur améliorer le côté actionnable du message sinon RAS (exemple et lien dans la doc https://github.com/ecolabdata/ecospheres-front/blob/main/CONTRIBUTING.md). |
Nouvelle version avec l'alerte en format tuile, mais comme elle est seule dans la liste, j'ai l'impression que cela écrase le message. J'ai fait une première évolution du message d'erreur en suggérant de créer un bouquet. Pour éviter la confusion entre un role perçu comme différent du bouton et du lien vers "créer un bouquet", j'ai préférer retirer le premier. |
Je trouve ça clean sans le bouton. Mais je suis pas convaincu par le format tuile. Vu qu'il n'y a pas d'autres tuiles pour comparer, ça fait étriqué. Si le texte ne change pas, attention typo : théme -> thème |
This reverts commit 756c05c.
Closes #116
Depends on #156
Depends on #162
Recherche de bouquets par theme et chantier
/!\ A relire après avoir mergé #162 et #156 pour pouvoir ne voir que les modifications qui lui sont propre car cette branche inclus leur commits
Job story
Depuis la page d'acceuil et la page bouquet, je peux filtrer par theme et par chantier
Additional elements of these changeset