entrouvert
/
combo
16
1
Fork 0
Code Issues Pull Requests 11 Releases Wiki Activity

dataviz: allow control of total display in tables (#85654) #233

Merged
vdeniaud merged 3 commits from wip/85654-dataviz-permettre-de-n-afficher into main 2024-04-16 10:52:00 +02:00
Conversation 2 Commits 3 Files Changed 7 +208 -23
vdeniaud commented 2024-02-13 17:35:44 +01:00
Owner
Copy Link

Sur mon chemin vers ce patch j'ai trouvé notamment que le tableau exporté en ODS était inversé par rapport au rendu html, donc j'inclus un commit pour corriger ça.

Aussi on gagne d'avoir les totaux dans cet export ODS (avant ils n'y étaient pas car ajoutés au moment du rendu pygal).

Sur mon chemin vers ce patch j'ai trouvé notamment que le tableau exporté en ODS était inversé par rapport au rendu html, donc j'inclus un commit pour corriger ça. Aussi on gagne d'avoir les totaux dans cet export ODS (avant ils n'y étaient pas car ajoutés au moment du rendu pygal).
vdeniaud added 1 commit 2024-02-13 17:35:44 +01:00
gitea/combo/pipeline/head There was a failure building this commit Details
5d27255c2c dataviz: allow control of total display in tables (#85654)
vdeniaud force-pushed wip/85654-dataviz-permettre-de-n-afficher from 5d27255c2c to 47c33694b3 2024-02-13 17:42:14 +01:00 Compare
vdeniaud force-pushed wip/85654-dataviz-permettre-de-n-afficher from 47c33694b3 to dc25a254c8 2024-02-14 11:58:57 +01:00 Compare
vdeniaud force-pushed wip/85654-dataviz-permettre-de-n-afficher from dc25a254c8 to 12a5d5c091 2024-02-14 12:00:12 +01:00 Compare
vdeniaud force-pushed wip/85654-dataviz-permettre-de-n-afficher from 12a5d5c091 to 53f204886f 2024-02-14 12:08:53 +01:00 Compare
vdeniaud reviewed 2024-02-14 13:16:04 +01:00
combo/apps/dataviz/forms.py Outdated
@ -217,6 +218,7 @@ class ChartNgForm(ChartFiltersMixin, forms.ModelForm):
'time_range_end',
'time_range_start_template',
'time_range_end_template',
'display_total',
vdeniaud commented 2024-02-14 12:09:34 +01:00
Author
Owner
Copy Link

Le nouveau champ n'apparaît pas pour bijoe

Le nouveau champ n'apparaît pas pour bijoe
combo/apps/dataviz/static/css/combo.chartngcell.css Outdated
@ -14,2 +14,4 @@
content: "\f019"; /* download */
}
.dataviz-table.total-line tr:last-child,
vdeniaud commented 2024-02-14 13:13:58 +01:00
Author
Owner
Copy Link

pygal ajoutait le total dans une balise <tfood> et ça se retrouvait pris dans un style posé par gadjo, donc ce bout pour conserver le rendu

pygal ajoutait le total dans une balise `<tfood>` et ça se retrouvait pris dans un style posé par gadjo, donc ce bout pour conserver le rendu
tests/test_dataviz.py Outdated
@ -1683,0 +1686,4 @@
@with_httmock(new_api_mock)
@pytest.mark.freeze_time('2023-09-28')
def test_chartng_cell_view_new_api_export_ods_total(app, normal_user, new_api_statistics):
vdeniaud commented 2024-02-14 13:15:49 +01:00
Author
Owner
Copy Link

Détour par l'export ODS pour tester tout ça, nettement plus clair que les tests qui font '<td>222</td>' in resp.text actuels

Détour par l'export ODS pour tester tout ça, nettement plus clair que les tests qui font `'<td>222</td>' in resp.text` actuels
vdeniaud changed title from WIP: dataviz: allow control of total display in tables (#85654) to dataviz: allow control of total display in tables (#85654) 2024-02-14 13:20:01 +01:00
fpeters approved these changes 2024-04-16 10:21:01 +02:00
fpeters left a comment
Owner
Copy Link

J'aurais mis "totals" plutôt que "total" mais go, au pire dans la trad on peut écrire totaux.

J'aurais mis "totals" plutôt que "total" mais go, au pire dans la trad on peut écrire totaux.
vdeniaud force-pushed wip/85654-dataviz-permettre-de-n-afficher from 53f204886f to 6be1d6c5fc 2024-04-16 10:32:06 +02:00 Compare
vdeniaud commented 2024-04-16 10:36:19 +02:00
Author
Owner
Copy Link

J'ai un biais contre le mot « totaux » que j'ai toujours trouvé chelou, j'avais réfléchi à une formulation des options où on pouvait écrire total au singulier dans le libellé du champ, mais en vrai je n'y attacherai aucune importance :)

J'ai un biais contre le mot « totaux » que j'ai toujours trouvé chelou, j'avais réfléchi à une formulation des options où on pouvait écrire total au singulier dans le libellé du champ, mais en vrai je n'y attacherai aucune importance :)
fpeters commented 2024-04-16 10:50:26 +02:00
Owner
Copy Link

J'ai un biais contre le mot « totaux » que j'ai toujours trouvé chelou, j'avais réfléchi à une formulation des options où on pouvait écrire total au singulier dans le libellé du champ, mais en vrai je n'y attacherai aucune importance :)

Ok je te laisserai la trad tu pourras mettre "total" dedans.

> J'ai un biais contre le mot « totaux » que j'ai toujours trouvé chelou, j'avais réfléchi à une formulation des options où on pouvait écrire total au singulier dans le libellé du champ, mais en vrai je n'y attacherai aucune importance :) Ok je te laisserai la trad tu pourras mettre "total" dedans.
🎉 1
vdeniaud merged commit 6be1d6c5fc into main 2024-04-16 10:52:00 +02:00
vdeniaud deleted branch wip/85654-dataviz-permettre-de-n-afficher 2024-04-16 10:52:00 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/combo#233
No description provided.
Delete Branch "wip/85654-dataviz-permettre-de-n-afficher"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?