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.

Sign up for GitHub

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

Jump to bottom

Ajout, gestion et affichage des langues régionales #631

Merged
merged 52 commits into from
Jul 21, 2022
Merged

Ajout, gestion et affichage des langues régionales #631

merged 52 commits into from
Jul 21, 2022

Conversation

Maxine-Le-Pennec
Copy link
Contributor

@Maxine-Le-Pennec Maxine-Le-Pennec commented Jul 12, 2022 •
edited
Loading

Cette PR fait suite au ticket suivant : Ajouter des champs texte alternatifs pour les langues régionales #625

La section "Nom" des formulaires d’édition des voies et des toponymes propose à présent un bouton Ajouter une langue régionale permettant d'ajouter un champ texte supplémentaire afin d'indiquer une alternative du nom.

Ce nouveau champ texte est accompagné :

  • d’un sélecteur pour indiquer la langue régionale utilisée pour cette alternative (Liste des langues supportées)
  • d’un bouton "corbeille" permettant de supprimer le champ
  • du bouton "assistance à la saisie des accents"

Si le formulaire est validé alors qu'un champ "nom alternatif" est vide, alors le formulaire ignore cette langues régionales.

Les noms alternatifs des voies et des toponymes sont affichés dans la liste des voies/toponymes, mais aussi dans l'en-tête de leurs pages.

Le premier nom alternatif figure sous le nom principal avec une police d’écriture plus petite et en italique.
Si plus de deux noms alternatifs existent, alors la mention Afficher les alternatives régionales sera affichée sous le nom principal avec une police d’écriture plus petite et en italique. Le survole de ce texte fait apparaitre un tooltip qui liste tous les noms alternatifs.

VOIES

Capture d’écran 2022-07-11 à 16 14 39

Capture d’écran 2022-07-11 à 16 46 09

Capture d’écran 2022-07-11 à 16 46 36

TOPONYMES

Capture d’écran 2022-07-11 à 16 53 02

Capture d’écran 2022-07-11 à 16 53 20

@Maxine-Le-Pennec Maxine-Le-Pennec self-assigned this Jul 12, 2022
@Maxine-Le-Pennec Maxine-Le-Pennec changed the title Maxine/langues regionales Ajout , gestion et affichage des langues régionales Jul 13, 2022
@Maxine-Le-Pennec Maxine-Le-Pennec changed the title Ajout , gestion et affichage des langues régionales Ajout, gestion et affichage des langues régionales Jul 13, 2022
@Maxine-Le-Pennec Maxine-Le-Pennec marked this pull request as ready for review July 13, 2022 09:47
cedric-famibelle-pronzola
cedric-famibelle-pronzola suggested changes Jul 13, 2022
View reviewed changes
Copy link
Contributor

@cedric-famibelle-pronzola cedric-famibelle-pronzola left a comment

Choose a reason for hiding this comment

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

En sélectionnant une langue, le sélecteur reste ouvert.

Après avoir cliqué sur "Ajouter une langue régionale", on peut enregistrer après avoir rempli le champ, sans avoir sélectionné de langue.

components/bal/language-field.js Outdated Show resolved Hide resolved
components/bal/language-field.js Outdated Show resolved Hide resolved
components/bal/toponyme-editor.js Outdated Show resolved Hide resolved
GllmR
GllmR suggested changes Jul 15, 2022
View reviewed changes
Copy link
Contributor

@GllmR GllmR left a comment

Choose a reason for hiding this comment

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

Lorsque l’on sélectionne une langue, il faudrait que le panneau se ferme automatiquement.

Les noms de voies en langues régionales n’appariassent pas dans l’onglet "toponymes"

Avec le "ajouter une adresse", il n’est pas possible d’ajouter un nom de voie alternatif.

Dans "numéros", il n’est pas possible d'ajouter un nom de voie alternatif lors de la création d’une nouvelle voie.

La taille des textes a augmentée -> Je ne pense pas que ce soit la bonne PR pour le faire et :

image

@Maxine-Le-Pennec
Copy link
Contributor Author

Maxine-Le-Pennec commented Jul 15, 2022 •
edited
Loading

Lorsque l’on sélectionne une langue, il faudrait que le panneau se ferme automatiquement.

Corrigé dans le commit af00546

Les noms de voies en langues régionales n’appariassent pas dans l’onglet "toponymes"

Corrigé dans le commit 117a87f

  • Avec le "ajouter une adresse", il n’est pas possible d’ajouter un nom de voie alternatif.
  • Dans "numéros", il n’est pas possible d'ajouter un nom de voie alternatif lors de la création d’une nouvelle voie.

À voir avec Théo pourquoi on ne doit pas les ajouter un nom de voie alternatif depuis le composant de création d'adresse.

La taille des textes a augmentée -> Je ne pense pas que ce soit la bonne PR pour le faire et :

La taille augmente afin d'obtenir un design harmonieux avec l'ajout des noms alternatifs.

image

Corrigé dans le commit bd2d110

@Maxine-Le-Pennec Maxine-Le-Pennec force-pushed the Maxine/langues-regionales branch from bd2d110 to 11024b2 Compare July 15, 2022 15:58
tmerlier
tmerlier suggested changes Jul 18, 2022
View reviewed changes
components/assisted-text-field.js Outdated Show resolved Hide resolved
components/assisted-text-field.js Outdated Show resolved Hide resolved
components/bal/language-field.js Outdated Show resolved Hide resolved
components/bal/language-field.js Outdated Show resolved Hide resolved
components/bal/language-field.js Outdated Show resolved Hide resolved
components/bal/voie-editor.js Outdated Show resolved Hide resolved
components/bal/numero-editor/numero-voie-selector.js Outdated Show resolved Hide resolved
components/bal/numero-editor.js Outdated Show resolved Hide resolved
components/bal/toponyme-editor.js Outdated Show resolved Hide resolved
components/bal/toponyme-editor.js Outdated Show resolved Hide resolved
jdesboeufs
jdesboeufs requested changes Jul 21, 2022
View reviewed changes
package.json Outdated Show resolved Hide resolved
scripts/available-flags.js Outdated Show resolved Hide resolved
@jdesboeufs jdesboeufs temporarily deployed to mes-adresses-maxine-lan-gbtnr4 July 21, 2022 11:59 Inactive
@jdesboeufs jdesboeufs temporarily deployed to mes-adresses-maxine-lan-gbtnr4 July 21, 2022 12:12 Inactive
@jdesboeufs jdesboeufs temporarily deployed to mes-adresses-maxine-lan-gbtnr4 July 21, 2022 13:32 Inactive
@jdesboeufs jdesboeufs temporarily deployed to mes-adresses-maxine-lan-gbtnr4 July 21, 2022 14:22 Inactive
@jdesboeufs jdesboeufs mentioned this pull request Jul 21, 2022
Closed
3 tasks
@tmerlier tmerlier merged commit 1db0c64 into master Jul 21, 2022
@tmerlier tmerlier deleted the Maxine/langues-regionales branch July 21, 2022 14:54
@tmerlier tmerlier mentioned this pull request Jul 21, 2022
Closed
1 task
@tmerlier tmerlier mentioned this pull request Jul 29, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GllmR GllmR GllmR approved these changes

@cedric-famibelle-pronzola cedric-famibelle-pronzola cedric-famibelle-pronzola approved these changes

@tmerlier tmerlier tmerlier approved these changes

@jdesboeufs jdesboeufs jdesboeufs approved these changes

Assignees

@Maxine-Le-Pennec Maxine-Le-Pennec

Labels
do-not-merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Maxine-Le-Pennec @jdesboeufs @tmerlier @cedric-famibelle-pronzola @GllmR