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

Bloque le chargement automatique des iframes #6610

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

Situphen
Copy link
Member

@Situphen Situphen commented Apr 23, 2024

PR liée au sujet sur le forum concernant l'intégration des sources externes

J'ai trouvé un moyen d'ajouter une bannière de consentement pour les contenus externes sans devoir modifier le code HTML généré par zmarkdown. L'idée est d'extraire les <iframe> du DOM dès que celui-ci est chargé. Ça fonctionne plutôt bien en local mais il faudrait voir ce que ça donne sur la bêta.

Inconvénients : c'est du bricolage ; ce n'est pas forcément efficace à 100% ; une fois mis en place, on risque de ne pas avoir la motivation de travailler à une meilleure solution

Avantages : pas besoin de modifier zmarkdown (donc pas besoin d'attendre plusieurs semaines ou mois) ; pas besoin de régénérer les anciens contenus (donc pas besoin d'attendre plusieurs années qu'ils soient mis à jour)

Il me faudra de l'aide sur le CSS car j'ai essayé de faire un truc propre pendant une bonne demie-heure sans succès puis j'ai laissé tombé.

Je veux bien aussi de l'aide sur le code JS pour vérifier que je n'utilise pas une syntaxe trop récente (qui ne fonctionnerait donc pas sur la majorité des navigateurs).

QA : (Lorsque ce sera déployé sur la bêta)

  • Créer un contenu avec toutes les intégrations externes possibles (les différents hébergeurs vidéos supportés ainsi que JSFiddle)
  • Vérifier qu'il n'y a pas de requêtes vers ces services externes avec les outils de développement intégrés au navigateur et une extension type uBlock Origin
  • Vérifier le bon fonctionnement du bouton pour charger l'intégration externe
  • Vérifier les deux derniers points sur plusieurs navigateurs (à minima Firefox et Chrome) sur ordinateur et sur mobile

@Situphen Situphen added this to En développement in Suivi des PR via automation Apr 23, 2024
@coveralls
Copy link

Coverage Status

coverage: 88.752%. remained the same
when pulling 44637fc on Situphen:iframe-consent
into 7b70da3 on zestedesavoir:dev.

@AmauryCarrade AmauryCarrade self-assigned this Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Suivi des PR
  
En développement
Development

Successfully merging this pull request may close these issues.

None yet

3 participants