search

epfl-si / jahiap

4 stars 1 forks source link

Supprimer les javascripts du template non-responsive #284

Closed williambelle closed 6 years ago

williambelle commented 7 years ago

Il faudrait supprimer les javascripts du template non-responsive :

https://github.com/epfl-idevelop/container-wp-volumes/blob/e824b6839b6f38e274652261d53df1f95fa0d6a5/wp-content/themes/epfl/header.php#L22-L24

LuluTchab commented 6 years ago

Vu qu'on est sur un thème responsive maintenant, je considère que cette issue n'est plus d'actualité.

williambelle commented 6 years ago

Je ne suis pas d'accord. Il y a un mélange entre le theme non-responsive et le theme responsive.

Dans la situation actuelle, jQuery EPFL est chargé deux fois, la bannière est créée deux fois (donc deux cliques pour accepter) et les données Google Analytics collectées deux fois...

screen

Selon moi, il faut supprimer ces 3 lignes !!!

LuluTchab commented 6 years ago

T'as raison. De mémoire, ces 3 lignes étaient nécessaires pour le header (sinon il ne réagissait pas correctement) mais ça devait effectivement être lié encore au thème non responsive.

LuluTchab commented 6 years ago

@alinekeller Je me permets de te l'assigner sans passer par Hicham car c'est vraiment un truc super rapide à faire.