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

Itinéraires / Ajouter un champs texte "Matériel" #2845

Closed
camillemonchicourt opened this issue Dec 7, 2021 · 7 comments
Closed

Itinéraires / Ajouter un champs texte "Matériel" #2845

camillemonchicourt opened this issue Dec 7, 2021 · 7 comments

Comments

@camillemonchicourt
Copy link
Member

Après "Recommandations".
Ajouter à l'API.

LePetitTim added a commit that referenced this issue Jan 11, 2022
LePetitTim added a commit that referenced this issue Jan 11, 2022
@camillemonchicourt
Copy link
Member Author

camillemonchicourt commented Jan 11, 2022

Pour le logo de ce champs dans le PDF, on souhaite utiliser un sac à dos, quelque quelque chose comme :

A voir si ça colle avec le style des autres pictos.

@camillemonchicourt
Copy link
Member Author

J'ai mis le picto ici : https://geotrek.ecrins-parcnational.fr/picto-equipment.svg

LePetitTim added a commit that referenced this issue Jan 12, 2022
LePetitTim added a commit that referenced this issue Jan 14, 2022
LePetitTim added a commit that referenced this issue Jan 14, 2022
LePetitTim added a commit that referenced this issue Jan 14, 2022
LePetitTim added a commit that referenced this issue Jan 14, 2022
LePetitTim added a commit that referenced this issue Jan 14, 2022
@camillemonchicourt
Copy link
Member Author

camillemonchicourt commented Feb 2, 2022

Ajouté dans la 2.76.0.
Traduit "Equipement" en français, mais je pense que le terme "Matériel" avait été retenu et à utiliser.
Mais je comprends que c'est parce que c'est le terme utilisé dans Sites Outdoor, donc c'est surement pour la cohérence.
Je vérifie.

  • Il a été intégré dans le PDF public, mais avec un S, et sans le picto évoqué plus haut. Et mettre à minima le titre du bloc en gras ("Equipement" au singulier, potentiellement à renommer "Matériel")

Capture d’écran de 2022-02-03 00-54-39

@camillemonchicourt
Copy link
Member Author

camillemonchicourt commented Feb 3, 2022

Alors pour le champs "Matériel", j'ai regardé pour mieux comprendre.
Dans Outdoor, on avait en effet 2 champs :

  • Matériel (gear) correspondant au matériel conseillé au pratiquant
  • Equipement (equipment) correspondant aux équipements du parcours gérés par le gestionnaire de celui-ci

Dans le module Itinéraire, c'est bien uniquement la notion de "Matériel" que l'on veut proposer à la saisie. La notion d'équipement étant géré par ailleurs dans les modules de gestion.

  • Dans ITINERAIRES, il faut bien que le nouveau champs soit nommé "Matériel" et non pas "Equipement". C'est vrai qu'en anglais "matériel" est souvent traduit "equipment", mais pour la cohérence avec le module Outdoor et éviter les confusions, je resterai bien sur le terme "gear" pour ce champs dans ITINERAIRES, en anglais, mais aussi au niveau BDD et API, si il est encore temps de revenir dessus.

@camillemonchicourt
Copy link
Member Author

camillemonchicourt commented Feb 7, 2022

  • OK dans la 2.76.1, le champs gear (décliné par langue) a été intégré à la table trekking_trek.
    Par contre j'ai toujours le champs ``equipment` (décliné par langue). Il faudrait le supprimer.
    Ceux qui passeront directement à la version 2.76.1 sans installer la 2.76.0 n'auront pas ce champs ?

  • La documentation de customisation des champs parle encore de equipment au niveau de COLUMNS_LISTS["trek_view"], COLUMNS_LISTS["trek_export"] et HIDDEN_FORM_FIELDS["trek"]. C'est bien à remplacer par gear ici aussi ?

  • Dans l'export PDF, le picto du champs MATERIEL a un soucis de taille et le titre du champs n'a pas été dissocié (en gras et en couleur) :

Capture d’écran de 2022-02-07 11-02-41

@camillemonchicourt
Copy link
Member Author

Dans l'export PDF, le style du picto et du titre "Matériel" n'était pas pris en compte car sur mon serveur de test, j'avais une surcouche du style CSS des PDF randos avec un fichier /opt/geotrek-admin/var/conf/extra_templats/trekking/trek_public_pdf.css.

En le supprimant (ou en y répercutant les évolutions CSS liées à l'ajout du champs "Matériel"), c'est OK :
Capture d’écran de 2022-02-09 11-10-00

Par contre, petit détail, mais tous les titres des champs sont en vert, mais celui-ci a été mis en orange. Je pense qu'il faut le mettre en vert comme les autres.

@camillemonchicourt
Copy link
Member Author

Retours intégrés dans la 2.76.3.
Reste la couleur dans le PDF qui est mystérieuse au niveau du CSS du PDF qui devrait tous être orange.

A voir par ailleurs :

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

No branches or pull requests

1 participant