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

correctif : ETQ instructeurs, lorsque je sors du composant de filtrage sans choisir une option, le site renvoie une erreur #9839

Conversation

mfo
Copy link
Contributor

@mfo mfo commented Dec 11, 2023

sentry: https://demarches-simplifiees.sentry.io/issues/4496045049/?project=1429550&query=&referrer=issue-stream&statsPeriod=14d&stream_index=3

Au passage, pour les combobox qui tappent sur des APIs, j'en profite pour mettre une loupe plutot qu'un chevron (qui fait croire que c'est un dropdown, remonté par Andy des fonds vert sur la config Chorus)
Screenshot 2023-12-12 at 9 49 51 AM

@mfo mfo force-pushed the US/fix-filter-focus-out-with-not-selected-filter branch 3 times, most recently from d952799 to 786d553 Compare December 12, 2023 09:05
@mfo mfo changed the title WIP - Us/fix filter focus out with not selected filter correctif : ETQ instructeurs, lorsque je sors du composant de filtrage sans choisir une option, le site renvoie une erreur Dec 12, 2023
@mfo mfo force-pushed the US/fix-filter-focus-out-with-not-selected-filter branch from 4758e5f to 786d553 Compare December 12, 2023 14:30
Copy link
Member

@colinux colinux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reste quelques détails, mais ça me parait bien solutionner le pb

.scss-lint.yml Outdated Show resolved Hide resolved
app/components/dsfr/combobox_component.rb Show resolved Hide resolved
app/components/dsfr/combobox_component.rb Outdated Show resolved Hide resolved
@mfo mfo force-pushed the US/fix-filter-focus-out-with-not-selected-filter branch 6 times, most recently from 2d26aef to e375095 Compare December 13, 2023 16:37
Copy link
Member

@colinux colinux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cf commentaire sur l'icône/css.

Sinon j'ai une remarque sur le filtre ETQ autocomplete, qui n'est pas lié à cette PR mais je la squatte pour expliquer. Je me rends compte que le scénario suivant n'est pas du tout pratique et je serai curieux de voir si on a des retours :

  • je sélectionne un filtre dans l'autocomplete
  • en fait je me suis trompé, je veux le changer en recliquant sur le chevron
  • ne s'affiche plus la liste de tous les filtres (car la sélection en cours a été filtrée par le premier choix).

Donc moi j'ai percuté (au bout d'un moment) qu'il faut supprimer la valeur saisie avant de refaire mon choix pour faire apparaître la liste, mais je ne trouve pas ça intuitif, d'autant que :

  • le chevron fait qu'on s'attend à un comportement de type select
  • parfois l'autocomplete matche 1 ou 2 autres filtres ce qui affiche une liste limitée à 3-4 choix.

Bref c'est inhérent à l'autocomplete, mais je demande si on devrait pas permettre une option pour ces cas là qui réafficherait toute la liste comme à l'état initial quand on clique sur le chevron.
Capture d’écran 2023-12-14 à 10 24 07
Capture d’écran 2023-12-14 à 10 19 00

app/assets/stylesheets/dsfr_override.scss Outdated Show resolved Hide resolved
@mfo mfo force-pushed the US/fix-filter-focus-out-with-not-selected-filter branch 2 times, most recently from aa8331f to 48f320d Compare December 14, 2023 10:09
@mfo
Copy link
Contributor Author

mfo commented Dec 14, 2023

Je pense comprendre ton point, pour rephraser on voudrait faire comme un <select> normal :

Tu clique, ça ouvre les options.
Tu input, ca oriente sur l'option la plus proche
Tu selectionne, ca ferme.
Quand tu reclique, ca affiche la liste avec ton option highligthée/centré ds la dialog.

@mfo mfo force-pushed the US/fix-filter-focus-out-with-not-selected-filter branch from 48f320d to 3d394b5 Compare December 14, 2023 13:09
@colinux
Copy link
Member

colinux commented Dec 14, 2023

Exactement. Avec ce comportement je me dis qu' on pourrait le mettre à pas mal d'endroits

Martin and others added 4 commits December 15, 2023 11:23
…dden], utilise cette option pour autosubmit uniquement un input [dans notre cas, ne soumet pas a l'input du combobox, mais a l'input hidden une fois le combobox selectionné]
Co-authored-by: Colin Darie <colin@darie.eu>
@mfo mfo force-pushed the US/fix-filter-focus-out-with-not-selected-filter branch from 3d394b5 to b539fe1 Compare December 15, 2023 10:23
@colinux colinux added this pull request to the merge queue Dec 21, 2023
Merged via the queue into demarches-simplifiees:main with commit e17c3bd Dec 21, 2023
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants