feat: rework search filter order #597
Conversation
Volubyl
force-pushed
the
596-rework-filters-layout
branch
from
efba7d3
to
df6e5e6
Compare
Volubyl
force-pushed
the
596-rework-filters-layout
branch
from
1e49eaa
to
3bf267a
Compare
Volubyl
force-pushed
the
596-rework-filters-layout
branch
8 times, most recently
from
c807576
to
07c9641
Compare
|
@johanricher la refonte de la page /search est sur staging. Tu peux faire joujou avec et me dire si ça te convient ? |
|
J'ai un comportement étrange avec le filtre "mot-clé" uniquement (à priori), quand un mot-clé est sélectionné et que je clique sur "réinitialiser le filtre" ça me sort de la recherche. A priori c'est une régession liée à cette PR car je n'ai pas ce comportement sur la prod. Cependant, après avoir vérifié que tu arrives à le reproduire, si tu préfères l'isoler dans un ticket dédié, on peut s'y attaquer après avoir mergé. |
|
j'arrive à repoduire je check cet aprem |
Volubyl
force-pushed
the
596-rework-filters-layout
branch
from
0afc360
to
8556025
Compare
|
c'est bon c'est fix @johanricher |
|
Top ça marche nickel. A part les 2 erreurs typographiques que j'ai commentées ci-dessus, c'est bon pour moi pour merger ! 👍 |
|
@johanricher tu as commenté où ? |
| value: "myorganization", | ||
| }, | ||
| geographicalCoverage: { | ||
| key: "Couverture géopgraphique", |
| @@ -8,11 +8,13 @@ | |||
| export let buttonText: string | null = ""; | |||
| export let label: string; | |||
| export let options: SelectOption<any>[]; | |||
| export let higlighted = false; | |||
| export let selected = false; | |||
|
|
|||
| $: options = [ | |||
| { | |||
| label: "Réinistialiser le filtre", | |||
|
Autant pour moi, les commentaires étaient "pending" j'avais pas cliqué sur le bouton pour qu'ils apparaissent en review ! |
close #596
close #592
closes #585
closes #577
Cette PR
Cette PR revoit l'ordre d'apparition des filtre de recherche