zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 161 forks source link

Bloque le chargement automatique des iframes #6610

Open Situphen opened 7 months ago

Situphen commented 7 months ago

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)

coveralls commented 7 months ago

Coverage Status

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