Bug #3120
Mettre moins en évidence le numéro de dossier informatique.
100%
Description
Le supprimer ou l'afficher en plus petit.
Files
Associated revisions
facturation: gray act ids on invoicing page
Closes #3120
History
Updated by Frédéric Péters about 10 years ago
- File 0001-do-not-display-n-de-dossier-informatique-3120.patch 0001-do-not-display-n-de-dossier-informatique-3120.patch added
- Patch proposed set to Yes
Patch qui le supprime.
Updated by Frédéric Péters about 10 years ago
- Status changed from Nouveau to En cours
- % Done changed from 0 to 50
% mis pour indiquer qu'il y a un patch.
Updated by Frédéric Péters about 10 years ago
Je ne vois pas le commit http://repos.entrouvert.org/calebasse.git/log/ (?)
Updated by Serghei Mihai about 10 years ago
C'est dans mon dépot local pour l'instant.
Je vais pousser aujourd'hui.
Updated by Mikaël Ates about 10 years ago
- Status changed from Résolu (à déployer) to En cours
- % Done changed from 100 to 0
Le patch supprime l'affichage du numéro informatique sur le premier onglet du dossier patient ce qui répond partiellement à la demande (incomplète).
Le numéro informatique apparaît à plusieurs endroits dans le logiciel mais ce n'est pas au niveau du premier onglet du dossier patient que cela peut apporter une confusion à l'utilisateur notamment puisqu'il est indiqué "numéro informatique" devant le numéro.
L'idée ce serait plutôt de ne pas l'afficher lorsque cela n'est pas nécessaire et lorsqu'on l'affiche le mettre en grisé. Ainsi il n'y a pas de confusion possible sur le numéro affiché.
Il faudrait donc le restaurer sur le premier onglet du dossier patient et le griser.
L'affichage du numéro informatique a été supprimé a plusieurs endroits dans l'agenda. Il reste à le supprimer sur la page de validation des actes.
Dans les statistiques il n'apparaît pas.
Dans la partie dossier, dans le dossier cf plus haut. Sur les tableaux il est en libellé de la colonne. On laisse ainsi ou on grise le contenu de la colonne.
Dans la saisie des actes et la facturation, il est indiqué entre parenthèse. On grise les parenthèses et le numéro.
Updated by Serghei Mihai about 10 years ago
- File 0001-patient-record-id-grayed-on-all-pages-in-order-to-av.patch 0001-patient-record-id-grayed-on-all-pages-in-order-to-av.patch added
- Assignee set to Serghei Mihai
- % Done changed from 0 to 50
Modifications apportées.
Updated by Jérôme Schneider about 10 years ago
OK pour moi.
Juste dans calebasse/agenda/templates/agenda/act-validation.html il faudrait remplacer :
{% if acte.patient.paper_id %} {{ acte.patient.paper_id }} {% endif %}
par :
{{ acte.patient.paper_id|default_if_none:"" }}Updated by Serghei Mihai about 10 years ago
- File 0001-patient-record-id-grayed-on-all-pages-in-order-to-av.patch 0001-patient-record-id-grayed-on-all-pages-in-order-to-av.patch added
Normalement il n'est pas censé de passer le if is le paper_id est None.
En revanche j'ai rajouté le default_if_none dans le listing des actes.
Updated by Serghei Mihai about 10 years ago
- Status changed from En cours to Résolu (à déployer)
- % Done changed from 50 to 100
Appliqué par commit calebasse|commit:1f4ae3b347f15c0133b00b968bfc1ad9aab281d5.
Updated by Mikaël Ates about 10 years ago
- Status changed from Résolu (à déployer) to En cours
"Dans la saisie des actes et la facturation, il est indiqué entre parenthèse. On grise les parenthèses et le numéro." C'est bon pour la saisie des actes mais pas pour la facturation.
Updated by Mikaël Ates about 10 years ago
Le template est sûrement à factoriser mais cette modif est à appliquer 6 fois dans le fichier:
diff --git a/calebasse/facturation/templates/facturation/detail.html b/calebasse/facturation/templates/facturation/detail.html
index 058c051..4b6eb48 100644
--- a/calebasse/facturation/templates/facturation/detail.html
+++ b/calebasse/facturation/templates/facturation/detail.html
@@ -311,8 +311,8 @@
{% endif %}
{% endif %}
- <h3><a href="../../dossiers/{{ patient.pk }}/view" target="_blank">
- {% if patient.paper_id %}{{ patient.paper_id }} {% endif %}({{ patient.pk }}) <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
+ <h3><span class="lightgray">({{ patient.pk }})</span>
+ <a href="../../dossiers/{{ patient.pk }}/view" target="_blank">{% if patient.paper_id %}{{ patient.paper_id }} {% endif %} <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
<div class="info">
</div>
{% if patient.pause %}
@@ -385,8 +385,8 @@
<div id="dossiers-concernes">
{% for patient, detail in patients_stats %}
<div class="frame dossier" id="to-hide">
- <h3>{% if patient.pk %}<a href="../../dossiers/{{ patient.pk }}/view" target="_blank">{% endif %}
- {% if patient.paper_id %}{{ patient.paper_id }} {% endif %}{% if patient.pk %}({{ patient.pk }}) {% endif %}<span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}{% if patient.pk %}</a>{% endif %}<
+ <h3><span class="lightgray">({{ patient.pk }})</span>
+ <a href="../../dossiers/{{ patient.pk }}/view" target="_blank">{% if patient.paper_id %}{{ patient.paper_id }} {% endif %} <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
<div class="info">
</div>
<ul>
@@ -435,8 +435,8 @@
{% else %}
<div class="frame dossier">
{% endif %}
- <h3><a href="../../dossiers/{{ patient.pk }}/view" target="_blank">
- {% if patient.paper_id %}{{ patient.paper_id }} {% endif %}({{ patient.pk }}) <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
+ <h3><span class="lightgray">({{ patient.pk }})</span>
+ <a href="../../dossiers/{{ patient.pk }}/view" target="_blank">{% if patient.paper_id %}{{ patient.paper_id }} {% endif %} <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
<div class="info">
</div>
{% if patient.pause %}
@@ -474,8 +474,8 @@
<div id="dossiers-concernes">
{% for patient, detail in patients_stats %}
<div class="frame dossier" id="to-hide">
- <h3><a href="../../dossiers/{{ patient.pk }}/view" target="_blank">
- {% if patient.paper_id %}{{ patient.paper_id }} {% endif %}({{ patient.pk }}) <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
+ <h3><span class="lightgray">({{ patient.pk }})</span>
+ <a href="../../dossiers/{{ patient.pk }}/view" target="_blank">{% if patient.paper_id %}{{ patient.paper_id }} {% endif %} <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
<div class="info">
</div>
<ul>
@@ -512,8 +512,8 @@
{% else %}
<div class="frame dossier">
{% endif %}
- <h3><a href="../../dossiers/{{ patient.pk }}/view" target="_blank">
- {% if patient.paper_id %}{{ patient.paper_id }} {% endif %}({{ patient.pk }}) <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
+ <h3><span class="lightgray">({{ patient.pk }})</span>
+ <a href="../../dossiers/{{ patient.pk }}/view" target="_blank">{% if patient.paper_id %}{{ patient.paper_id }} {% endif %} <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
<div class="info">
</div>
{% if patient.pause %}
@@ -561,8 +561,8 @@
<div id="dossiers-concernes">
{% for patient, detail in patients_stats %}
<div class="frame dossier" id="to-hide">
- <h3><a href="../../dossiers/{{ patient.pk }}/view" target="_blank">
- {% if patient.paper_id %}{{ patient.paper_id }} {% endif %}({{ patient.pk }}) <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
+ <h3><span class="lightgray">({{ patient.pk }})</span>
+ <a href="../../dossiers/{{ patient.pk }}/view" target="_blank">{% if patient.paper_id %}{{ patient.paper_id }} {% endif %} <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
<div class="info">
</div>
<ul>
Updated by Frédéric Péters about 10 years ago
Il est plutôt conseillé d'ajouter de la sémantique au niveau des classes, plutôt qu'une description graphique (i.e. class="patient-pk" plutôt que class="lightgray").
Updated by Serghei Mihai about 10 years ago
- File 0001-facturation-gray-act-ids-on-invoicing-page.patch 0001-facturation-gray-act-ids-on-invoicing-page.patch added
Remarques prises en compte.
Je prefere les "_" dans les noms des classes css.
Updated by Thomas Noël about 10 years ago
Serghei Mihai a écrit :
Je prefere les "_" dans les noms des classes css.
Heuula ... https://developer.mozilla.org/en-US/docs/Web/Guide/CSS/Underscores_in_class_and_ID_Names
Updated by Serghei Mihai about 10 years ago
- Status changed from En cours to Résolu (à déployer)
Appliqué par commit calebasse|commit:3c1f0b4a59e4052b3c0bf3bfb8d84ee716d6f685.
patient record id grayed on all pages in order to avoid confusion
Closes #3120