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

wip/65947-Cellule-Graph-Pouvoir-exporter-l (#65947) #165

Merged
vdeniaud merged 2 commits from wip/65947-Cellule-Graph-Pouvoir-exporter-l into main 2023-11-14 10:43:56 +01:00
Conversation 3 Commits 2 Files Changed 12 +204 -13
vdeniaud commented 2023-09-28 15:18:36 +02:00
Owner
Copy Link

Bouton positionné en bas à gauche, affiché quand la souris est sur la cellule.

Bouton positionné en bas à gauche, affiché quand la souris est sur la cellule.
1695915329.png
20 KiB
1695915329.png
vdeniaud force-pushed wip/65947-Cellule-Graph-Pouvoir-exporter-l from 54c80c3d4b to 80293a1f4a 2023-09-28 17:11:38 +02:00 Compare
vdeniaud changed title from WIP: wip/65947-Cellule-Graph-Pouvoir-exporter-l (#65947) to wip/65947-Cellule-Graph-Pouvoir-exporter-l (#65947) 2023-09-28 17:39:52 +02:00
fpeters approved these changes 2023-10-27 14:43:44 +02:00
Dismissed
fpeters left a comment
Owner
Copy Link

J'aurais bien vu le téléchargement d'ods aussi quand c'est un graphe qui est affiché, mais ok.

J'aurais bien vu le téléchargement d'ods aussi quand c'est un graphe qui est affiché, mais ok.
combo/apps/dataviz/templates/combo/chartngcell.html Outdated
@ -42,0 +52,4 @@
download{% if not cell.is_table_chart %}="{{ cell.get_download_label }}.svg"{% endif %}
title="{% trans "Download" %}"
aria-label="{% trans "Download" %}">
</a>
fpeters commented 2023-10-27 14:42:35 +02:00
Owner
Copy Link

Pour l'accessibilité je pense que c'est mieux d'avoir <span class="sr-only">{% trans "Download" %}</span> (plutôt que l'aria-label), ne pas avoir d'élément vide. (mais je n'ai pas de référence pour affirmer ça), tu changes si tu veux.

Pour l'accessibilité je pense que c'est mieux d'avoir `<span class="sr-only">{% trans "Download" %}</span>` (plutôt que l'aria-label), ne pas avoir d'élément vide. (mais je n'ai pas de référence pour affirmer ça), tu changes si tu veux.
vdeniaud commented 2023-10-30 18:16:55 +01:00
Author
Owner
Copy Link

Fait, merci

Fait, merci
vdeniaud force-pushed wip/65947-Cellule-Graph-Pouvoir-exporter-l from 80293a1f4a to bc85aa15b5 2023-10-30 09:29:00 +01:00 Compare
vdeniaud force-pushed wip/65947-Cellule-Graph-Pouvoir-exporter-l from bc85aa15b5 to 079692ad12 2023-10-30 18:02:27 +01:00 Compare
vdeniaud force-pushed wip/65947-Cellule-Graph-Pouvoir-exporter-l from 079692ad12 to 54e89a70cf 2023-10-30 18:16:39 +01:00 Compare
vdeniaud commented 2023-10-30 18:19:57 +01:00
Author
Owner
Copy Link

J'aurais bien vu le téléchargement d'ods aussi quand c'est un graphe qui est affiché, mais ok.

Je suis tout à fait pour mais je suis en panne d'inspi pour l'interface, puisque dans le ticket tu as exclu « une popup qui permet de choisir le format csv ou autre », tu penses à un deuxième bouton avec une icône qui fait penser à un tableau ?

... Et en jouant en local je m'aperçois que mon astuce de position est foireuse, ça ne marche pas quand il y a plusieurs cellules, je remets en WIP.

> J'aurais bien vu le téléchargement d'ods aussi quand c'est un graphe qui est affiché, mais ok. Je suis tout à fait pour mais je suis en panne d'inspi pour l'interface, puisque dans le ticket tu as exclu « une popup qui permet de choisir le format csv ou autre », tu penses à un deuxième bouton avec une icône qui fait penser à un tableau ? ... Et en jouant en local je m'aperçois que mon astuce de position est foireuse, ça ne marche pas quand il y a plusieurs cellules, je remets en WIP.
vdeniaud dismissed fpeters’s review 2023-10-30 18:20:30 +01:00
Reason:

Il y a un bug

vdeniaud changed title from wip/65947-Cellule-Graph-Pouvoir-exporter-l (#65947) to WIP: wip/65947-Cellule-Graph-Pouvoir-exporter-l (#65947) 2023-10-30 18:20:47 +01:00
vdeniaud changed title from WIP: wip/65947-Cellule-Graph-Pouvoir-exporter-l (#65947) to wip/65947-Cellule-Graph-Pouvoir-exporter-l (#65947) 2023-10-31 16:12:40 +01:00
vdeniaud commented 2023-10-31 16:14:04 +01:00
Author
Owner
Copy Link

Ouf ça n'était pas trop compliqué de corriger le positionnement du bouton, par contre je ne mesure pas les impacts de passer le div des cellules graphe en position: relative, si c'est OK je squasherai.

Ouf ça n'était pas trop compliqué de corriger le positionnement du bouton, par contre je ne mesure pas les impacts de passer le div des cellules graphe en position: relative, si c'est OK je squasherai.
vdeniaud requested review from fpeters 2023-10-31 16:14:14 +01:00
vdeniaud commented 2023-11-02 11:21:23 +01:00
Author
Owner
Copy Link

J'aurais bien vu le téléchargement d'ods aussi quand c'est un graphe qui est affiché, mais ok.

Je suis tout à fait pour mais je suis en panne d'inspi pour l'interface, puisque dans le ticket tu as exclu « une popup qui permet de choisir le format csv ou autre », tu penses à un deuxième bouton avec une icône qui fait penser à un tableau ?

Vu à l'oral, on va passer par une popup.

> > J'aurais bien vu le téléchargement d'ods aussi quand c'est un graphe qui est affiché, mais ok. > > Je suis tout à fait pour mais je suis en panne d'inspi pour l'interface, puisque dans le ticket tu as exclu « une popup qui permet de choisir le format csv ou autre », tu penses à un deuxième bouton avec une icône qui fait penser à un tableau ? Vu à l'oral, on va passer par une popup.
vdeniaud changed title from wip/65947-Cellule-Graph-Pouvoir-exporter-l (#65947) to WIP: wip/65947-Cellule-Graph-Pouvoir-exporter-l (#65947) 2023-11-02 11:21:29 +01:00
vdeniaud force-pushed wip/65947-Cellule-Graph-Pouvoir-exporter-l from 0861645c91 to ede25eab10 2023-11-02 17:38:47 +01:00 Compare
vdeniaud changed title from WIP: wip/65947-Cellule-Graph-Pouvoir-exporter-l (#65947) to wip/65947-Cellule-Graph-Pouvoir-exporter-l (#65947) 2023-11-02 17:48:17 +01:00
fpeters approved these changes 2023-11-14 10:30:49 +01:00
fpeters left a comment
Owner
Copy Link

Mini commentaire mais je valide dès à présent.

Mini commentaire mais je valide dès à présent.
combo/apps/dataviz/models.py Outdated
@ -303,1 +305,4 @@
def get_download_label(self):
label = slugify(self.title or self.statistic.label)
return 'export-%s-%s' % (label, date.today().strftime('%Y%m%d'))
fpeters commented 2023-11-14 10:30:24 +01:00
Owner
Copy Link

C'est curieux que ça s'appelle get_download_label mais que ça ne retourne pas vraiment un libellé, je proposerais d'avoir get_download_filename(self, extension).

C'est curieux que ça s'appelle get_download_label mais que ça ne retourne pas vraiment un libellé, je proposerais d'avoir `get_download_filename(self, extension)`.
vdeniaud commented 2023-11-14 10:34:15 +01:00
Author
Owner
Copy Link

Yep, je fais la modif

Yep, je fais la modif
vdeniaud force-pushed wip/65947-Cellule-Graph-Pouvoir-exporter-l from ede25eab10 to 3eb771ddbd 2023-11-14 10:33:20 +01:00 Compare
vdeniaud merged commit 3eb771ddbd into main 2023-11-14 10:43:56 +01:00
vdeniaud deleted branch wip/65947-Cellule-Graph-Pouvoir-exporter-l 2023-11-14 10:43:56 +01: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#165
No description provided.
Delete Branch "wip/65947-Cellule-Graph-Pouvoir-exporter-l"

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?