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

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

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 080197b 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 ! ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Modification demandée
Development

Successfully merging this pull request may close these issues.

3 participants