Resolve "Journal de calcul repliable"

Closes #519 (closed)

changed milestone to %Version 4.15.1

added Feature suggestion labels

assigned to @francois.grand

By Grand Francois on 2022-06-14T14:07:01 (imported from GitLab)

added 2 commits

• c9e3fbe2 - feat: make result messages log foldable
• 9c398df6 - feat: add application setup option to fold result messages log by default

Compare with previous version

By Grand Francois on 2022-06-14T15:51:04 (imported from GitLab)

marked this merge request as ready

By Grand Francois on 2022-06-15T16:51:10 (imported from GitLab)

C'est super !

Est-ce qu'on peut mettre le titre en gras pour avoir la même police que les entêtes de tableau de résultat.

A priori, ça revient à changer la propriété font-weight de .mat-expansion-panel-header à 600 d'après mon test:

By Dorchies David on 2022-06-16T10:06:32 (imported from GitLab)

Pour répondre à ta question:

y'a aussi un journal dans le vérificateur. il faut aussi le modifier ?

Non, je ne pense pas. Ce journal c'est le résultat du calcul, il n'y a donc pas de raison de vouloir le cacher contrairement à d'autres modules où le journal de calcul peut occulter les résultats.

By Dorchies David on 2022-06-16T10:11:26 (imported from GitLab)

J'ai regardé les journaux composés d'une synthèse et d'un journal par calcul (exemples PAB et pré-barrage) et ça a l'air de bien marché.

Avec le titre en gras, je pense qu'on sera bon

By Dorchies David on 2022-06-16T10:17:10 (imported from GitLab)

added 1 commit

• 91f01bae - fix: set log panel title to bold

Compare with previous version

By Grand Francois on 2022-06-16T10:33:03 (imported from GitLab)

J'ai peur que l'OFB demande, à raison, à ce qu'il reste une information pertinente quand le journal est replié.

Je m'explique, quand le journal est replié, à fortiori par défaut, le fait qu'il y ait une erreur ou un avertissement ne saute pas aux yeux de l'utilisateur. Ce problème ne se pose que si on effectue un calcul avec des paramètres fixes. En effet, quand les paramètres varient les avertissements et erreurs sont signalés dans la première colonne du tableau de résultat ou dans la liste déroulante du choix de résultat à afficher.

@francois.grand Est-ce que tu penses qu'il serait opportun de changer le titre pour indiquer le nombre d'avertissements et d'erreurs qudn le journal est replié ?

By Dorchies David on 2022-06-16T14:34:17 (imported from GitLab)

added 2 commits

• e9ddaff7 - update jalhyd_branch
• 88a7166e - feat: foldable log: add error/warning/info summary in title

Compare with previous version

By Grand Francois on 2022-06-16T17:54:40 (imported from GitLab)

merged

mentioned in commit f014ec74

By Dorchies David on 2022-06-27T15:52:28 (imported from GitLab)