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

Fixes #6690: Configure le bon chemin pour les extensions MathJax #6691

Merged
merged 2 commits into from
Dec 15, 2024

Conversation

Situphen
Copy link
Member

@Situphen Situphen commented Dec 5, 2024

Depuis le passage de MathJax via CDN à MathJax en local (#6606), le script MathJax.js n'arrive pas à charger les fichiers complémentaires. Normalement, la détection de la racine devrait se faire automatiquement : le script est /static/js/MathJax.js donc les extensions se trouvent aussi dans /static/js. (Citation de la documentation : "MathJax locates the <script> tag that loaded the MathJax.js file, and sets the MathJax.Hub.config.root value to reflect the location of the MathJax root directory.") Ça ne fonctionne pas en production car le nom du script est hashé : MathJax.e380311aa78d.js au lieu de MathJax.js. Cela se corrige facilement, il suffit de définir explicitement le chemin dans la configuration comme le fait cette PR.

QA : Mettre cette PR sur la bêta et vérifier que les requêtes vers les fichiers complémentaires n'échouent pas.

@coveralls
Copy link

coveralls commented Dec 5, 2024

Coverage Status

coverage: 89.2%. remained the same
when pulling fda4219 on Situphen:fix-mathjax
into d38b46a on zestedesavoir:dev.

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ça ne fonctionne pas, il essaie toujours de charger https://beta.zestedesavoir.com/extensions/MathMenu.js?V=2.7.1.

C'est déployé sur la bêta.

@Situphen
Copy link
Member Author

Situphen commented Dec 7, 2024

C'est étonnant, ça fonctionne bien sur la bêta pour moi pourtant !
image

@philippemilink
Copy link
Member

Je ne comprends pas : une fois sur deux sur cette page https://beta.zestedesavoir.com/tutoriels/244/comment-rediger-des-maths-sur-zeste-de-savoir/ les deux scripts de MatJax sont en 404... J'ai testé différents trucs, mais maintenant je suis juste paumé entre ce qui fonctionne et ce qui ne fonctionne pas...

@Situphen
Copy link
Member Author

Situphen commented Dec 8, 2024

Ça fonctionnait correctement avec Firefox chez moi mais pas avec Google Chrome. C'était probablement dû à une différence dans l'ordre de chargement des scripts JS (le script MathJax était avant la configuration MathJax dans le code HTML mais avec un async). Normalement avec mon dernier commit ça corrige bien le soucis. Je te laisse mettre à jour la bêta et vérifier ! ;)

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ça a l'air bon maintenant.

Par contre puisque le fichier mathjax_config.html est utilisé uniquement pour être inclus dans templates/mathjax.html, autant mettre son contenu directement là où il est inclus, non ? Ça évite de lire un fichier supplémentaire sur le disque, ça soulage (très peu) le travail du moteur de templates.

@Situphen
Copy link
Member Author

C'est modifié !

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA OK ✔️

@philippemilink philippemilink merged commit 79a2033 into zestedesavoir:dev Dec 15, 2024
12 checks passed
@Situphen Situphen deleted the fix-mathjax branch December 15, 2024 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Fusionnée
Development

Successfully merging this pull request may close these issues.

3 participants