Refont UI/UX dashBoard.

[Benjamin-etu-lille]

Refonte UI/UX dashboard

IMPORTANT : Cette PR est la continuité de la PR #1296 : "Ajout de filtres à la recherche dynamique sur le dashboard".
Pour la revue, n'hésitez pas à consulter la PR fermée. Elle avait été clôturée car j'avais terminé ma période de stage.
Je pense qu'elle était proche d'être correcte, il serait donc dommage de l'abandonner.

IMPORTANT: This PR is a continuation of PR #1296: “Adding filters to dynamic search on the dashboard.”
For review, feel free to consult the closed PR. It was closed because my internship period had ended.
I think it was close to being correct, so it would be a shame to abandon it.

• You have read our contribution guidelines.
• Your PR targets the dev_v4 branch.
• Your PR status is in draft if it's still a work in progress.

[LoicBonavent]

Hello Benjamin,

Merci beaucoup pour tout ce travail de qualité.
Au vue de tout ce code, j'ai réalisé des tests sur un environnement de test.
Voici les erreurs que j'obtiens :

• problème dans la console du navigateur : filter_aside_video_list_refresh.js?ver=4.0.0.benj:210 Uncaught ReferenceError: filtersConfig is not defined
• avertissement dans la console du navigateur : non unique ids pour id_is_draft
• chargement infini des vidéos : les mêmes vidéos sont affichées en boucle quand je scrolle
• la recherche par type et par discipline ne fonctionne pas : cela affiche toujours qu'aucune vidéo ne correspond aux critères.
• Pour rester standard avec les autres libellés, il faudrait mettre Catégorie à la place de Catégories (je crois qu'Olivier l'avait identifié aussi).
  Merci encore

[LoicBonavent]

J'ai trouvé d'autres problèmes :

• une fois que l'on a Tout sélectionné, on ne peut pas Tout désélectionner
• en mode d'affichage par liste, quand on scrolle, cela affiche une partie de la vignette sur toute la ligne

Par contre, le fait de devoir sélectionner au moins une vidéo pour que les Actions multiples apparaissent me paraît perturbant. Ce système d'actions multiples est de plus en plus utilisé, et les gens se sont habitués à le voir par défaut. Je pense qu'il serait préférable de l'afficher en permanence, quitte à le réduire en hauteur un peu si c'est possible.

A propos, je teste sur Edge (si ça marche avec edge, ça marche partout :) ).
Merci

[Benjamin-etu-lille]

Merci pour vos retours. J'en profite pour vous demander : auriez-vous des recommandations ou des techniques particulières pour bien tester les nouvelles fonctionnalités d'une PR ?

[Badatos]

Merci pour vos retours. J'en profite pour vous demander : auriez-vous des recommandations ou des techniques particulières pour bien tester les nouvelles fonctionnalités d'une PR ?

mes recommandations : Cliquer absolument partout, et faire tester par une tierce personne ;)

[LoicBonavent]

Hello Benjamin,

Perso, je pars du principe que les tests unitaires ne servent qu'à éviter des régressions et ne servent finalement qu'à éviter les bourdes de codage, pas plus.
Je réalise par la suite des tests fonctionnels à la main, sans utilisation d'outils particuliers (excepté la console du navigateur), en général sur plusieurs navigateurs; dans ces tests fonctionnels, j'essaie de passer vraiment partout et avec différents profils (non connecté, personnel, administrateur...).

Cela serait bien d'avoir des outils automatisés qui permettent de réaliser ces tests fonctionnels (par exemple basé sur Selenium); mais à ce jour, rien n'a été fait dans ce sens. Après, ce n'est pas très grave : perso, je trouve que ces tests fonctionnels sont rapides, bien plus que de coder des modifications dans Pod...

Bonne journée

[LoicBonavent]

Merci pour ces modifications; cela résout certains problèmes, mais pas tous on dirait; il reste les problèmes suivants :

• la recherche par type et par discipline ne fonctionne pas : cela affiche toujours qu'aucune vidéo ne correspond aux critères.

• Pour rester standard avec les autres libellés, il faudrait mettre Catégorie à la place de Categories.

• une fois que l'on a Tout sélectionné, on ne peut pas Tout désélectionner

• afficher Actions multiples en permanence ?

Merci

[LoicBonavent]

AH oui, un petit détail; vis_à-vis de la terminologie Pod, plutôt que tag, mettre "mot clé".
Merci

[Benjamin-etu-lille]

Pour ce qui concerne d'afficher Actions multiples en permanence. Initialement, l'idée de ne les afficher qu'après sélection (validée avec un designer web) était pour éviter de surcharger l'interface lorsque aucune action n'est possible,

Cependant, je suis d'accord que la découvrabilité (jsp si ca exixte comme mot) de la fonction peut être améliorée. Plutôt que de l'afficher en permanence, que penses-tu d'ajouter un petit texte d'indication près de la case "Tout sélectionner" du type : "Sélectionnez des vidéos pour activer les actions multiples." ? Ce texte disparaîtrait dès qu'une vidéo est cochée.

[LoicBonavent]

Je ne pense pas que ce mot existe, mais il est très parlant :)
Oui, le petite texte d'indication me paraît très bien; merci beaucoup.

[LoicBonavent]

Super boulot. Merci beaucoup. Et bon week-end / bonnes vacances :)

[Badatos]

apparament, il y a une erreur révélée par les tests unitaires :

======================================================================
FAIL: test_get_videos_view (pod.video.tests.test_views.VideosTestView)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/Esup-Pod/Esup-Pod/pod/video/tests/test_views.py", line 604, in test_get_videos_view
    self.assertEqual(response.context["videos"].paginator.count, 3)
AssertionError: 1 != 3

[LoicBonavent]

Hello Benjamin,
Ta PR est bien finie ou il te reste encore des points à voir ?
Si elle est bien finie, on pourra faire un tour pour checker les dernières modifications.
N'hésite pas à nous tenir au courant.
Merci.

Bonne journée

[Benjamin-etu-lille]

Bonjour à tous, excusez-moi pour le délai de réponse. En effet, ma PR est bien terminée.

[LoicBonavent]

Hello Benjamin, super. merci beaucoup. Et cela a déjà été mergé, yes !