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

Améliore la vocalisation des situations particulières #367

Merged
merged 3 commits into from Sep 19, 2016

Conversation

MattiSG
Copy link
Member

@MattiSG MattiSG commented Aug 26, 2016

Fix #356.

Pour la redondance entre facultatif et autre, j'aurais tendance à penser qu'il vaut mieux retirer facultatif : autre joue aussi un rôle de placebo sur « je suis dans une autre situation particulière », qui ne serait pas tenu par facultatif.
Par ailleurs, c'est le seul endroit dans toute l'app où on précise facultatif, et ça implique donc que tous les autres cas où ça n'est pas précisé, ce n'est pas le cas… ce qui est faux.

WDYT?

@MattiSG
Copy link
Member Author

MattiSG commented Aug 26, 2016

Facultatif a été ajouté dans 5c2240e par @Flightan, apparemment au détour d'une modification technique. @Flightan, as-tu souvenir du raisonnement ayant mené à cet ajout ?

@fpagnoux
Copy link
Contributor

Sauf si @Flightan se souvient d'une très bonne raison pour ce "facultatif", je pense aussi qu'il faut le supprimer, vu la confusion que ça a créé lors du test.

C'est peut-être hors-scope de la PR, mais est-ce qu'on aurait pas aussi intérêt à remplacer l'intitulé de la question. Que pensez vous de :
image
(avec les points médians qu'il faut, of course ;))

J'ai l'impression que "situation particulière" n'a du sens que du point de vue du développeur, parce que pour le bénéficiaire potentiel il n'y a pas grand chose en commun entre être enceinte, étudiant, ou inapte au travail.

@MattiSG
Copy link
Member Author

MattiSG commented Aug 29, 2016

J'ai l'impression que "situation particulière" n'a du sens que du point de vue du développeur

Absolument. Et la formulation est incohérente par rapport aux autres types de question.

D'ailleurs, avec « vous êtes », je ne suis pas sûr du tout que « autre » soit encore nécessaire… :)

@Flightan
Copy link
Member

Je ne me souviens pas très bien du contexte du "facultatif", c'est un UX qui nous avait suggéré l'idée pour que nos utilisateurs ne pensent plus que le champs soit obligatoire. Je suis pour la suppression du facultatif et "Autre"

@MattiSG MattiSG assigned MattiSG, Flightan and fpagnoux and unassigned MattiSG Sep 6, 2016
@MattiSG
Copy link
Member Author

MattiSG commented Sep 12, 2016

On pourrait encore améliorer, refactor et rendre neutre en genre, ça ne serait pas du luxe. Mais on peut aussi déjà merger cette PR.

@@ -113,24 +113,24 @@

<div class="form-group from-group-lg">
<label class="control-label col-sm-4" id="situations-particulieres">
Situations particulières <em>(facultatif)</em>
Vous êtes
Copy link
Contributor

Choose a reason for hiding this comment

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

Et si ce n'est pas le formulaire du demandeur, mais de son conjoint ?

@MattiSG MattiSG assigned MattiSG and Flightan and unassigned Flightan, fpagnoux and MattiSG Sep 13, 2016
@MattiSG MattiSG force-pushed the situations-particulieres-356 branch 2 times, most recently from bd7893a to 34883cd Compare September 19, 2016 14:06
Copy link
Member

@Flightan Flightan left a comment

Choose a reason for hiding this comment

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

GTM malgré le label for=id avec l'input imbriqué redondant.

@MattiSG
Copy link
Member Author

MattiSG commented Sep 19, 2016

malgré le label for=id avec l'input imbriqué redondant.

Sera traité lors d'une PR ultérieure refactor 😉

@MattiSG MattiSG assigned MattiSG and unassigned Flightan Sep 19, 2016
@MattiSG MattiSG merged commit 8946bce into master Sep 19, 2016
@MattiSG MattiSG deleted the situations-particulieres-356 branch September 19, 2016 14:33
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.

Améliorer la saisie des situations particulières
4 participants