search

MaVoix / candidature

Soumission et publication des inscriptions au tirage au sort #MAVOIX
The Unlicense
4 stars 0 forks source link

Prénom composés / min / maj #42

Closed tielpi closed 7 years ago

tielpi commented 7 years ago

Quand les candidat.e.s ont des prénoms composés, le deuxième prénom est mis en minuscule et il est impossible de le corriger

ColinMaudry commented 7 years ago

Mmmh je pense que @oceanBigOne force la mise en minuscule des autres lettres, alors qu'il faut juste forcer la mise en majuscule de la première lettre.

oceanBigOne commented 7 years ago

Effectivement ! .. je suis sur un hotfix à ce sujet là au niveau de l'export (démarrer entre midi et deux, je reprend ce soir).

J'en profite pour uniformiser tout ça. Mais pour moi, il faut passer la première lettre de chaque mot en majuscule : "Jean Paul", "Aix En Provence" etc...
(il y a une différence de comportement dans les fonctions que j'utilise que je n'avais pas remarqué, désolé pour ça)

oceanBigOne commented 7 years ago

La règle exacte est ici : http://grammaire.reverso.net/5_2_01_la_majuscule_dans_les_noms_propres.shtml Je propose de rester sur ma proposition, à savoir, ne pas tenir compte de la notion de prépositions (en tout cas dans un premier temps) : "Aix En Provence" au lieu "Aix en Provence" . Est-ce que cela vous convient ?

ColinMaudry commented 7 years ago

Pour les prénoms OK de capitaliser chaque mot :+1:

oceanBigOne commented 7 years ago

Je viens de faire la modification pour les prénoms uniquement. A tester sur le serveur de dev avant (j'ai un doute sur la gestion de l'encodage). Il faut tester un un prénom commençant par un caractère accentué. genre "Élise"

ColinMaudry commented 7 years ago

Il semblerait que seule la première lettre de toute la chaîne soit mise en majuscule sur la page candidat, et les autres en minuscule. https://github.com/MaVoix/candidature/blob/develop/templates/visitor/candidature/candidat.html.twig#L52

On veut la première lettre de chaque mot.

oceanBigOne commented 7 years ago

Effectivement, un oubli sur cette page.

ColinMaudry commented 7 years ago

Mais il y a aussi une correction en entrée de base, ou juste à l'affichage ? Idéalement faudrait que le contenu en base soit nickel pour les nouvelles candidatures.

oceanBigOne commented 7 years ago

Pas trop d'avis sur la question ... je suis revenu en arrière sur ce point en me disant que c'était pas plus mal de conserver la saisie utilisateur "brut". Mais franchement je fais comme tu me dis (pour moi les deux visions se défendent et ça ne me pose aucun problème).

ColinMaudry commented 7 years ago

OK, on garde tel quel en entrée.