Bug #3120
Mettre moins en évidence le numéro de dossier informatique.
100%
Description
Le supprimer ou l'afficher en plus petit.
Fichiers
Révisions associées
facturation: gray act ids on invoicing page
Closes #3120
Historique
Mis à jour par Frédéric Péters il y a presque 10 ans
- Fichier 0001-do-not-display-n-de-dossier-informatique-3120.patch 0001-do-not-display-n-de-dossier-informatique-3120.patch ajouté
- Patch proposed mis à Oui
Patch qui le supprime.
Mis à jour par Frédéric Péters il y a presque 10 ans
- Statut changé de Nouveau à En cours
- % réalisé changé de 0 à 50
% mis pour indiquer qu'il y a un patch.
Mis à jour par Frédéric Péters il y a plus de 9 ans
Je ne vois pas le commit http://repos.entrouvert.org/calebasse.git/log/ (?)
Mis à jour par Serghei Mihai il y a plus de 9 ans
C'est dans mon dépot local pour l'instant.
Je vais pousser aujourd'hui.
Mis à jour par Serghei Mihai il y a plus de 9 ans
- Statut changé de En cours à Résolu (à déployer)
Mis à jour par Mikaël Ates il y a plus de 9 ans
- Statut changé de Résolu (à déployer) à En cours
- % réalisé changé de 100 à 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.
Mis à jour par Serghei Mihai il y a plus de 9 ans
- Fichier 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 ajouté
- Assigné à mis à Serghei Mihai
- % réalisé changé de 0 à 50
Modifications apportées.
Mis à jour par Jérôme Schneider il y a plus de 9 ans
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:"" }}Mis à jour par Serghei Mihai il y a plus de 9 ans
- Fichier 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 ajouté
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.
Mis à jour par Serghei Mihai il y a plus de 9 ans
- Statut changé de En cours à Résolu (à déployer)
- % réalisé changé de 50 à 100
Appliqué par commit calebasse|commit:1f4ae3b347f15c0133b00b968bfc1ad9aab281d5.
Mis à jour par Mikaël Ates il y a plus de 9 ans
- Statut changé de Résolu (à déployer) à 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.
Mis à jour par Serghei Mihai il y a plus de 9 ans
Mis à jour par Mikaël Ates il y a plus de 9 ans
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>
Mis à jour par Frédéric Péters il y a plus de 9 ans
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").
Mis à jour par Serghei Mihai il y a plus de 9 ans
- Fichier 0001-facturation-gray-act-ids-on-invoicing-page.patch 0001-facturation-gray-act-ids-on-invoicing-page.patch ajouté
Remarques prises en compte.
Je prefere les "_" dans les noms des classes css.
Mis à jour par Thomas Noël il y a plus de 9 ans
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
Mis à jour par Serghei Mihai il y a plus de 9 ans
- Statut changé de En cours à Résolu (à déployer)
Appliqué par commit calebasse|commit:3c1f0b4a59e4052b3c0bf3bfb8d84ee716d6f685.
patient record id grayed on all pages in order to avoid confusion
Closes #3120