[DONE] Fix : favorite js #848
Conversation
…o AymericJak/fix_favorite_js
pod/main/static/js/main.js
Outdated
|
|
||
| // Favorites | ||
|
|
||
| const favoriteButtonManage = document.querySelector("#card-managevideo #fav-button"); |
There was a problem hiding this comment.
je ne comprend pas. pourquoi retirer le bouton en js ?
There was a problem hiding this comment.
Le link_video.html relie à la fois la carte mais aussi la section sur la page video :
Donc je passe par le JS pour retirer uniquement le bouton si il est présent dans la section 'Gérer la vidéo'
There was a problem hiding this comment.
ok en regardant un peu merci de retirer ce code JS. Voici une alternative :
fichier video_edit.html L.166 {% include "videos/link_video.html" with video=form.instance hide_favorite_link=True %}
fichier link_video.html L.6 {% if USE_FAVORITES and not hide_favorite_link %}
Merci de tester et de me dire si c'est ok
There was a problem hiding this comment.
vous pouvez rajouter hide_favorite_link=True à tous les appels que vous souhaitez
There was a problem hiding this comment.
un peu mieux {% if USE_FAVORITES and not hide_favorite_link is True %}
There was a problem hiding this comment.
En modifiant ces 2 lignes :
{% include "videos/link_video.html" with video=form.instance hide_favorite_link=True %}{% if USE_FAVORITES and not hide_favorite_link is True %}
Le bouton apparait toujours pour les pages de complétion, cut...
There was a problem hiding this comment.
Non c'est bon, je sais !
Cannot read properties of null (reading 'addEventListener')