Skip to content
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

Merged
merged 21 commits into from
Dec 1, 2023
Merged

feat(nav): search by subtheme #168

merged 21 commits into from
Dec 1, 2023

Conversation

edelagnier
Copy link
Contributor

@edelagnier edelagnier commented Nov 17, 2023

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

Copy link

netlify bot commented Nov 17, 2023

Deploy Preview for ecospheres ready!

Name Link
🔨 Latest commit 9e68fd2
🔍 Latest deploy log https://app.netlify.com/sites/ecospheres/deploys/6569ab33ee8f1d0008975679
😎 Deploy Preview https://deploy-preview-168--ecospheres.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@edelagnier
Copy link
Contributor Author

Quelques differences avec la maquette :

Je n'ai pas mis les images et les tags dans la liste de tuiles :

  • tag , on utilise visiblement une version maison des Tiles qui ne les prends pas en charge, il faudra faire une issue a part pour l'inclure
  • image, a specifier mais je ne pense pas pertinent d'attendre pour livrer la fonctionnalite actuelle

@edelagnier
Copy link
Contributor Author

edelagnier commented Nov 17, 2023

A relire après avoir merger #162 et #156 pour pouvoir ne voir que les modifications qui lui sont propre car cette branche inclus leur commits

@bonjourmauko bonjourmauko changed the title Feature/search by subtheme feat(nav): search by subtheme Nov 17, 2023
@bonjourmauko bonjourmauko added the enhancement Améliorations et nouvelles fonctionnalités label Nov 17, 2023
@bonjourmauko bonjourmauko added this to the V1 : Découvrabilité par cas d’usage milestone Nov 17, 2023
Copy link
Contributor

@bonjourmauko bonjourmauko left a 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.

src/model.ts Outdated Show resolved Hide resolved
@edelagnier edelagnier force-pushed the feature/search_by_subtheme branch 2 times, most recently from 9c5b78c to dae6912 Compare November 20, 2023 09:14
@edelagnier edelagnier force-pushed the feature/search_by_subtheme branch from dae6912 to ee6d492 Compare November 20, 2023 09:19
@edelagnier
Copy link
Contributor Author

Manip git faite. La relecture est possible clairement maintenant

@edelagnier edelagnier requested a review from YeLnatSs November 20, 2023 09:29
.gitignore Outdated Show resolved Hide resolved
src/views/bouquets/BouquetsListView.vue Outdated Show resolved Hide resolved
src/model.ts Outdated Show resolved Hide resolved
src/components/TopicList.vue Outdated Show resolved Hide resolved
src/views/bouquets/BouquetsListView.vue Outdated Show resolved Hide resolved
@edelagnier edelagnier force-pushed the feature/search_by_subtheme branch from ee6d492 to 6a7af23 Compare November 22, 2023 08:57
@edelagnier edelagnier force-pushed the feature/search_by_subtheme branch from 2b53016 to 9858938 Compare November 22, 2023 14:23
@martyKN
Copy link
Contributor

martyKN commented Nov 22, 2023

@martyKN j'avais mis cette alerte en cas de liste de bouquet vide. c'est bon pour toi ?

image
dans l'idée c'est ok. mais je vois sur le dsfr qu'il y a un composant "alerte" qui fait la taille d'une tuile bouquet qui serait peut etre plus adapté

@edelagnier
Copy link
Contributor Author

Oh je ne l'avais pas vu je vais le changer alors

@bonjourmauko bonjourmauko self-requested a review November 23, 2023 10:01
@bonjourmauko
Copy link
Contributor

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).

@edelagnier
Copy link
Contributor Author

edelagnier commented Nov 29, 2023

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.
Peut etre pourrait on donner le contact de quelqu'un pouvant aider sur ce sujet ?

image

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.
@martyKN que suggérerais tu sur la forme et sur le message ?

@streino
Copy link
Contributor

streino commented Nov 29, 2023

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

tsconfig.json Outdated Show resolved Hide resolved
@edelagnier
Copy link
Contributor Author

typo corrigée.
en version non tuile et sans bouton, voici ce que ça donnerait :
image

@martyKN
Copy link
Contributor

martyKN commented Nov 29, 2023

typo corrigée. en version non tuile et sans bouton, voici ce que ça donnerait : image

pour moi c'est bon comme ca => étiré sans bouton

@bonjourmauko bonjourmauko mentioned this pull request Nov 29, 2023
streino
streino previously approved these changes Nov 29, 2023
bonjourmauko
bonjourmauko previously approved these changes Nov 29, 2023
@edelagnier edelagnier dismissed stale reviews from bonjourmauko and streino via 9e68fd2 December 1, 2023 09:45
@edelagnier edelagnier merged commit fffd57e into main Dec 1, 2023
4 checks passed
@bonjourmauko bonjourmauko deleted the feature/search_by_subtheme branch December 2, 2023 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Améliorations et nouvelles fonctionnalités
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(nav): découvrir les bouquets par chantier
4 participants