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

Update mistune (0.8 => 3.0) #707

Merged
merged 4 commits into from
Jul 8, 2024
Merged

Update mistune (0.8 => 3.0) #707

merged 4 commits into from
Jul 8, 2024

Conversation

imperosol
Copy link
Contributor

@imperosol imperosol commented Jul 8, 2024

Mise à jour de mistune vers sa version 3.

Ca permet en passant de retirer le Python <3.12 dans le pyproject.toml.

J'en ai aussi profité pour refaire un peu le style du rendu markdown, notamment en ce qui concerne les bouts de code et les blocs de citation.

On passe de ça :
image

A ça :
image

Le code inline change aussi :
image

la line-height des paragraphes, des ul et des ol a été légèrement augmentée.
Le code utilise la police Consolas (la police de Notepad, tmtc).

Les blocs de code et les blocs de citation sont un peu indentés, sauf sur petits écrans, où l'indentation est retirée.

Les tableaux ne prennent plus la totalité de l'espace (ce qui donnait des tableaux étrangement larges), mais seulement ce dont ils ont besoin.
image

J'ai rajouté du padding au-dessus des titres :
image

J'ai retiré la barre de progression, qui n'avait pas d'exemple et que j'ai pas réussi à faire marcher avec l'ancien code.

@imperosol imperosol requested a review from klmp200 July 8, 2024 14:10
klmp200
klmp200 previously approved these changes Jul 8, 2024
Copy link
Contributor

@klmp200 klmp200 left a comment

Choose a reason for hiding this comment

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

Ça m'a l'air bien
Je suis pas allé trop en détail mais j'ai rien vu de choquant
Et si ça fonctionne bien c'est cool :D
En plus ça rend vraiment bien :)

@imperosol
Copy link
Contributor Author

J'ai changé la règle pour les exposants et les indices dans la page du MDEditor, pour coller aux règles du parser du backend.

@imperosol imperosol requested a review from klmp200 July 8, 2024 15:09
@imperosol imperosol merged commit 7fe4951 into taiste Jul 8, 2024
3 checks passed
@Hyask
Copy link
Contributor

Hyask commented Jul 8, 2024

Bien joué aussi sur celle là, c'est chouette :-)

@imperosol imperosol deleted the update-mistune branch July 9, 2024 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants