Bloque le chargement automatique des iframes #6610
Draft
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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)