search

CAUCA-9-1-1 / survip

SURVI-Prevention Desktop
2 stars 3 forks source link

Survey texte format and table setting cannot be configured for reporting #495

Closed jocelyncardinal closed 5 years ago

jocelyncardinal commented 5 years ago

Describe the bug Survey texte format and table setting cannot be configured for reporting

Expected behavior Tag "@Inspection.SurveyAnswers@" should not generate "hardcoded" HTML text, table and/or any format setting.

Screenshots Here, a test with texts (and the tag "@Inspection.SurveyAnswers@") in RED inside a table with defined width ... Capture d’écran 2019-03-12 à 11.07.24.png

.... the result when générate the report ... Capture d’écran 2019-03-12 à 11.10.33.png

iamkinetic commented 5 years ago

C'est la seule section de la version de Julien des rapports qui a survécu dans son état à peu près originale.

Tu voudrais pouvoir faire quelque chose du genre?

@group.name@ @group.description@ @answer.question@ : @answer.answer@

C'est plus facile à dire qu'à faire parce que notre structure de questionnaire est très flexible et l'arbre des questions pourrait en théorie avoir autant de niveau qu'on voudrait (même si on limite présentement le niveau maximal à 2).

Je pense que je vais commencer à réfléchir à permettre l'utilisation d'un logiciel externe pour les rapports parce qu'on est déjà pas mal à la limite de ce qui me parait réaliste dans un outil custom fait dans CkEditor.

Je crois qu'on pourrait peut-être se permettre d'avoir quelque part dans le site où on pourrait configurer l'utilisation de ce logiciel externe. CAUSE a les licences nécessaires pour faire un éditeur de rapport en utilisant la librairie XtraReport (de DevExpress) qui est celle que nous utilisons à l'interne. Avec ce outil, nous aurions beaucoup plus de flexibilité dans l'édition des rapports. C'est à réfléchir.

jocelyncardinal commented 5 years ago

Pour l'instant, le besoin est beaucoup plus modeste - Seulement que le PLACEHOLDER "@Inspection.SurveyAnswers@" utilise le format texte défini en mode design (comme les autre PLACEHOLDER font déjà). Il s'agit probablement de simplement enlever les tags HTML 'Harcodé" par Julien dans le HTML produit par le PLACEHOLDER (dans mon exemple, ce qui n'est pas en rouge). En meme temps, enlever la largeur du tableau qui doit etre aussi "hardcodé" (WITDH=) ...

iamkinetic commented 5 years ago

OK, pour récapituler:

jocelyncardinal commented 5 years ago

Oui, pas avoir un style par défaut. Toutefois, je propose de conserver le BOLD pour le texte des groupes et sous-groupes (c'est deja comme ca) - ca devrait donner un résultats pas pire

Oui, ne pas avoir de largeur par défaut. Mettre une largeur à 100% ca me semble une bonne idée.

iamkinetic commented 5 years ago

Sera fait via #514.

iamkinetic commented 5 years ago

Ça donne ceci en pseudo html:

<h2>Catégorie</h2>
<h3>Sous-Catégorie</h3>
<table width=100%>
   <tr>
     <td width:35%>Question</td>
     <td width:65%>Réponse</td>
   </tr>
</table

À l'oeil c'est pas mal pareil comme avant, sauf que les largeurs sont en % au lieu d'être en dimensions fixes (c'était 8.5 pouces).

image.png