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

Le rendu des maths est cassé #5111

Closed
pierre-24 opened this issue Nov 19, 2018 · 15 comments · Fixed by #5131
Closed

Le rendu des maths est cassé #5111

pierre-24 opened this issue Nov 19, 2018 · 15 comments · Fixed by #5131
Labels
Bloquant Ticket qui doit être traité avant la prochaine mise à jour C-Front Concerne l'interface du site S-Régression Corrige un problème sur un composant qui fonctionnait auparavant

Comments

@pierre-24
Copy link
Member

Bonsoir,

Après la mise à jour du markdown qui avait un peu perturbé les formules, voici que le rendu des maths est cassé. Regardez un peu :

$$ \mu \cdot \mathrm{d}l \cdot \frac{\partial^2 z}{\partial t^2 }(x,t) = T \cdot \sin\theta(x+\mathrm{d} x,t) - T \cdot \sin\theta(x,t) ~~~~~~~~~~~~~(1) $$

$$ \int_0^1 \exp x \mathrm{d} x $$

On remarque différents problèmes :

  • toutes les lettres sont en romain au lieu de l'italique (?!),
  • les barres de fraction n'apparaissent plus (??!!).

J'ai l'impression que la tripaille derrière les formules a changé, du moins chez moi (Firefox 63 sous Linux Mint). En cliquant droit, je n'ai plus le petit menu sympathique où je peux visualiser le code LaTeX par exemple.

Heureusement, ça n'affecte pas les contenus publiés, mais c'est assez critique pour les auteurs de contenus scientifiques, honnêtement. Je cherche d'où ça vient exactement, mais si quelqu'un à la réponse, je suis preneur pour qu'on puisse corriger au plus vite.

Édit. : j'ai l'impression que les styles de KaTeX sont mal configurés, mais comme je connais mal, j'ai du mal à trouver pourquoi.

Sujet : https://zestedesavoir.com/forums/sujet/11615/le-rendu-des-maths-est-casse/
Envoyé depuis Zeste de Savoir

@pierre-24 pierre-24 added C-Front Concerne l'interface du site S-Régression Corrige un problème sur un composant qui fonctionnait auparavant Bloquant Ticket qui doit être traité avant la prochaine mise à jour labels Nov 19, 2018
@artragis
Copy link
Member

je ne comprends pas non plus nous n'avons pas changé la version de katex, car pour l'instant la 0.10 casse le rendu.

Après on put toujours voir mais bon.

@Arnaud-D
Copy link
Contributor

Arnaud-D commented Nov 21, 2018

Merci pour avoir remonté mon rapport de bug ici. :-)

J'ai fait quelques tests :

  1. Cela ne semble pas venir de KaTeX en soi, puisque que mon exemple fonctionne avec le rendu du site de KaTeX. Testé avec Chrome et Firefox, dans des versions récentes.
  2. Sur la bêta, le rendu semble correct : exemple.
  3. Il y a des différences sur le style de la bêta et celui de la prod pour la même formule. Par exemple :
    • sur la bêta, rendu correct (notez le mathit) :
      <span class="mord mathit" style="margin-right:0.13889em;">T</span>
    • sur la prod, rendu incorrect (notez le mathdefault) :
      <span class="mord mathdefault" style="margin-right:0.13889em;">T</span>

Je penche donc sur un problème de configuration du serveur de prod ou quelque chose du même acabi.

Je continue à chercher, mais je pense que je vais bientôt être limité par ma méconnaissance du fonctionnement de ZdS.

@artragis
Copy link
Member

Raah, ça ressemble vachement à un pb avec zmarkdown ping @vhf ça ressemble àc e qu'il se passe sur ma PR avec l'upgrade de rehype-katex

@vhf
Copy link
Contributor

vhf commented Nov 21, 2018

Quand ça : zestedesavoir/zmarkdown#297 sera réglé, je déploierai sur la beta et si c'est bon sur la beta je mettrai ça en prod.

@Arnaud-D
Copy link
Contributor

Comment peut-on avoir un problème de zmarkdown sur la prod et pas sur la bêta ?

@vhf
Copy link
Contributor

vhf commented Nov 22, 2018

@Arnaud-D La beta est souvent en avance sur la prod.

@vhf
Copy link
Contributor

vhf commented Nov 23, 2018

Déployé sur la beta. Mettez le CSS pour katex 0.10 sur la beta, allez tester là-bas et dites-moi si les maths sont correctes (katex 0.10). Il y a aussi des correctifs pour les tables ( zestedesavoir/zmarkdown#290 ).

Si c'est bon je déploie en prod.

@artragis
Copy link
Member

ok, on va devoir faire une 27.4 pour ça.

@pierre-24
Copy link
Member Author

Alors, je sais pas si c'est nous ou côté KaTeX, mais sur ce message ... On devrait normalement avoir des lignes horizontales et verticales dans le tableau, comme démontré . En particulier, le dernier tableau du message est un copier-collé de l'exemple suivant:

screenshot from 2018-11-23 15-32-44

Quand on regarde le code HTML, on voit qu'il y a bien des délimiteur (<span class="vertical-separator" ...> par exemple) qui sont ajouté. Je supose un problème de CSS :)

@pierre-24
Copy link
Member Author

(au cas ou ça serait lié, Firefox/63.0, Ubuntu 18.10).

@artragis
Copy link
Member

non mais ça marchera pas pour l'instant : on n' pas mis katex à jour sur la béta côté front. Je m'occupe de brancher et de faire ça ce soir.

@artragis
Copy link
Member

salut tout le monde

si vous pouviez passer sur la béta pour tester les maths histoire qu'on puisse s'assurer que le nouveau katex fonctionne bien.

@Arnaud-D
Copy link
Contributor

À première vue, ça marche bien pour moi.

@artragis
Copy link
Member

ok, merci.
@vhf et @sandhose vous pouvez vous coordonner pour mettre à jour zmd sur la prod et mettre en place la v27.4? une fois que ça sera fait, je ferai un cherry-pick pour la branche master

@vhf
Copy link
Contributor

vhf commented Nov 28, 2018

Yep, ici il est plus simple de déléguer que coordonner.

@sandhose Lors du déploiement il faudrait faire ceci:

## as root

cd /opt/zmd
# on sait jamais
mv package.json package.json.bak
mv package-lock.json package-lock.json.bak

wget https://gist.githubusercontent.com/vhf/4036bfad7da9b5702d5a9d04b07f82e3/raw/f44f32d4ac114e9fd08b08847f6b54ed0119435f/package-lock.json
wget https://gist.githubusercontent.com/vhf/4036bfad7da9b5702d5a9d04b07f82e3/raw/f44f32d4ac114e9fd08b08847f6b54ed0119435f/package.json

npm install
service zmd reload

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bloquant Ticket qui doit être traité avant la prochaine mise à jour C-Front Concerne l'interface du site S-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants