Skip to content

Conversation

@cen-cgeier
Copy link
Contributor

#127
Ajout de valeurs par défaut dans les différentes sections en cas d'absence d'informations en base de données.
Fichiers concernés par les modifications :

  • backend/gn_module_zh/templates/evaluation.html
  • backend/gn_module_zh/templates/fonctionnement.html
  • backend/gn_module_zh/templates/milieux.html
  • backend/gn_module_zh/templates/statuts.html
  • backend/gn_module_zh/templates/usages.html
  • backend/gn_module_zh/templates/delimitation.html

@edelclaux
Copy link
Contributor

ça a l'air niquel, je testerai bientôt

@edelclaux edelclaux self-assigned this Oct 16, 2025
@edelclaux
Copy link
Contributor

Une petite remarque à la marge:
image

Ta branche part de main. Soit tu l'as effectivement fait partir de main, soit tu l'as fait partir d'une ancienne version de develop.
Dans les deux cas, il vaut mieux mettre à jour ton develop, puis faire partir ta branche de la.

Bon, je dis ça, mais ici, ça ne doit pas changer grand chose.

@cen-cgeier
Copy link
Contributor Author

Je l'ai en effet fait partir de main. Il est préférable de la faire partir de develop ? Il faut que je change ?
Il me semble que main et develop sont au même niveau.

@edelclaux
Copy link
Contributor

Sur le contenu, c'est plutot pas mal.

Je trouve que ce serait plus sympa si c'était centré, avec une police légèrement plus claire (genre #333, quelque chose comme ça).
Pour centrer, il faut:

  • centrer le texte dans la cellule text-align: center;
  • faire que la cellule occupe toutes les colonnes
    exemple si trois colonnes:

Pour la validation du contenu, est-ce @B1234j ça te parait ok ?
Je met avec ce message un exemple de fiche PDF générée, avec toutes les sections non renseignées.
test_PR.pdf

@cen-cgeier
Copy link
Contributor Author

cen-cgeier commented Oct 17, 2025

Il me semble que le texte est déjà centré dans la cellule de sa colonne. C'est juste que la cellule prend une seule colonne au lieu de fusionner toutes les colonnes de la ligne du tableau.
Je me suis fait la même réflexion que toi sur le texte au centre du tableau mais je n'ai pas su fusionner les cellules d'une seule ligne du tableau. Après coup j'ai fini par trouver que tel que tu le présentes ce n'est pas déconnant, ni choquant. Est-ce que cela vaut le coup d'essayer d'aller plus loin sur le sujet ?

Concernant la couleur des valeurs ajoutées par défaut, j'ai pas d'avis pour le moment.

@edelclaux
Copy link
Contributor

Je l'ai en effet fait partir de main. Il est préférable de la faire partir de develop ? Il faut que je change ? Il me semble que main et develop sont au même niveau.

Non, pas besoin de changer pour cette fois, c'est une modif marginale.
Mais de manière générale, il vaut mieux partir de develop.

@edelclaux
Copy link
Contributor

Il me semble que le texte est déjà centré dans la cellule de sa colonne. C'est juste que la cellule prend une seule colonne au lieu de fusionner toutes les colonnes de la ligne du tableau. Je me suis fait la même réflexion que toi sur le texte au centre du tableau mais je n'ai pas su fusionner les cellules d'une seule ligne du tableau. Après coup j'ai fini par trouver que tel que tu le présentes ce n'est pas déconnant, ni choquant. Est-ce que cela vaut le coup d'essayer d'aller plus loin sur le sujet ?

Concernant la couleur des valeurs ajoutées par défaut, j'ai pas d'avis pour le moment.

ah oui, tiens, j'ai pas mis l'exemple dans ma réponse:

<!-- source du tableau ci-après -->
<!-- illustration du colspan -->
 <table>
  <tr>
   <td> Colonne 1 </td>
   <td> Colonne 2 </td>
  </tr>
  <tr>
   <td colspan="2"> Colonnes 1 et 2 </td>
  </tr>
 </table>
Colonne 1 Colonne 2
Colonnes 1 et 2

@cen-cgeier
Copy link
Contributor Author

Ha oui, simple ! J'avais pas trouvé.
Pourquoi pas ! A voir le rendu le plus agréable, j'essaie de mon côté fin de semaine pour donnée mon avis.

@cen-cgeier
Copy link
Contributor Author

Le rendu est top, Merci !

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