Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DONE] Feature : Add favorite videos #820

Conversation

SebastienCozeDev
Copy link
Collaborator

@SebastienCozeDev SebastienCozeDev commented Apr 24, 2023

  • Ajouter l'affichage du compteur de favoris.
  • Ajouter la page des vidéos favorites.
  • Ajouter les détails du nombre de favoris dans les statistiques de la vidéo (USE_STATS_VIEW).
  • Ajouter l'option USE_FAVORITES.
  • Ajouter la fonctionnalité pour pouvoir mettre en favoris.
  • Ajouter le drag and drop dans la page de favoris pour que l'utilisateur puisse ranger les vidéos. Peut-être avec Muuri ou SortableJS.
  • Ajouter les traductions.
  • Ajouter la possibilité de trier les vidéos favorites en fonction de la date d'ajout ou du rang.

@AymericJak
Copy link
Collaborator

AymericJak commented May 15, 2023

c'est encore moi sweat_smile Il faut vérifier la pagination car je n'ai pas l'impression que ça fonctionne correctement

Je viens de régler ce soucis.
Lors du clic sur le bouton réorganiser, le scroll infini se stoppe.

Copy link
Contributor

@ptitloup ptitloup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok pour moi, merci !

Copy link
Collaborator

@Badatos Badatos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je serai d'avis de mettre également le bouton "retirer des favoris" sur la liste des vidéos.

  • autres remarques ci-dessous :

pod/favorite/static/js/video-favorites-card-list.js Outdated Show resolved Hide resolved
pod/video/templates/videos/link_video.html Outdated Show resolved Hide resolved
pod/video/templates/videos/video-info.html Show resolved Hide resolved
pod/video/templates/videos/video-info.html Show resolved Hide resolved
pod/video/templates/videos/video-info.html Show resolved Hide resolved
Copy link
Collaborator

@Badatos Badatos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bravo pour le cadre, c'est bien visuel, on comprend mieux où va se placer la vidéo qu'on réorganise !
Plus que quelques petites corrections et ca devrait le faire ;)

pod/favorite/static/js/video-favorites-card-delete.js Outdated Show resolved Hide resolved
pod/locale/fr/LC_MESSAGES/django.po Outdated Show resolved Hide resolved
pod/locale/fr/LC_MESSAGES/django.po Outdated Show resolved Hide resolved
pod/locale/fr/LC_MESSAGES/django.po Outdated Show resolved Hide resolved

.shake-effect:active {
cursor: grabbing;
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:active ne fonctionne pas pour détecter un élément en cours de déplacement.
Je pense qu'il faut ajouter via JS une classe CSS à l'objet quand on commence le drag, et le retirer quand c'est terminé.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je ne vois pas de soucis avec ça, lorsque l'on maintient la video, le cursor: grabbing fonctionne parfaitement.
Puis le cursor se met bien à jour en fonction de si :

  • nous sommes sur une autre video : grabbing avec un petit plus.
  • nous ne sommes pas sur une vidéo : grabbing normal

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En effet, sur Firefox (113) le curseur "grabbing" s'affiche bien, par contre sur Safari (16.4) ce n'est pas le cas.
Sur chrome (Version 113), le curseur "grabbing" s'affiche furtivement, et est rapidement remplacé par le curseur "+", sans aucun grab ni grabbing (ca donne l'impression qu'on va ajouter la vidéo, voir la dupliquer)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

j'envoie une copie d'écran sur Rocket ;)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah ok, j'étais sur chrome [Ubuntu].

J'ai pas testé sur Safari :(

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Normalement, c'est fix ;)

Copy link
Collaborator

@Badatos Badatos May 16, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alors je viens de tester et :

  • Sur Safari : pas de changement, grab au survol puis curseur classique pendant le drag.
  • Sur Firefox : le curseur reste toujours en mode "grab", qu'on survole ou qu'on drag.
  • Sur Chrome : on a simplement perdu le "grabbing", le reste n'a pas changé :/

@ptitloup ptitloup merged commit 49d9d35 into EsupPortail:develop May 16, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants