Skip to content

Use MathML for math formatting#155

Draft
MatthijsBlom wants to merge 10 commits into
learnyouahaskell:mainfrom
MatthijsBlom:math-formatting
Draft

Use MathML for math formatting#155
MatthijsBlom wants to merge 10 commits into
learnyouahaskell:mainfrom
MatthijsBlom:math-formatting

Conversation

@MatthijsBlom
Copy link
Copy Markdown
Collaborator

@MatthijsBlom MatthijsBlom commented Apr 8, 2026

For now this is only a proof of concept.
If this direction is appreciated, I might look into it further.

To Do:

  • test on a variety of clients
  • find a more suitable math font
Before After

@pierluc-codes
Copy link
Copy Markdown
Collaborator

I like it!

For few second, I was hesitant because the previous syntax was closer to code. But, we are talking math, so the visual make sense.

If you can upload it somewhere, I can test on my iDevice and check if render similar.

Thanks!

Copy link
Copy Markdown
Collaborator

@ulysses4ever ulysses4ever left a comment

Choose a reason for hiding this comment

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

Great contribution, thanks!

Small comments inline.

Comment thread source_md/recursion.md Outdated
Comment thread source_md/starting-out.md
Comment thread source_md/higher-order-functions.md
@MatthijsBlom
Copy link
Copy Markdown
Collaborator Author

If you can upload it somewhere, I can test on my iDevice and check if render similar.

This branch is now live at matthijsblom.github.io/learnyouahaskell.github.io.

Need I fear that the changed build script (to be reverted before merge) on this branch will cause spurious builds in this repo? We don't want this branch served at learnyouahaskell.github.io.

@ulysses4ever
Copy link
Copy Markdown
Collaborator

Off-topic: We really should implement automatic PR preview. But last time I looked at it recently, is a bit of a pain in the neck to set up.

@pierluc-codes
Copy link
Copy Markdown
Collaborator

This branch is now live at matthijsblom.github.io/learnyouahaskell.github.io.

All good on my iDevice. Thank you!

@pierluc-codes
Copy link
Copy Markdown
Collaborator

@MatthijsBlom

If you can revert the github action change, remove the draft then I will approve. Thank you!

Copy link
Copy Markdown
Collaborator

@ulysses4ever ulysses4ever left a comment

Choose a reason for hiding this comment

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

LGTM modulo the workflow edits to deploy locally

Comment thread source_md/higher-order-functions.md
@ulysses4ever ulysses4ever self-requested a review May 1, 2026 21:58
@MatthijsBlom
Copy link
Copy Markdown
Collaborator Author

MatthijsBlom commented May 3, 2026

I think I've covered the whole book now.

I don't think the $…$-ification of single names is a big improvement. But using `…` or *…* isn't great either, nor is not styling at all. My preference for $…$ is very mild. In any case, let's be consistent.

I'll rebase to improve the commit messages and remove the CI change after incorporating the final feedback.

( By the way: I've been doing a pass through the whole book. If you're curious, see my dump branch. Not everything on there is meant for eventual merging. )

@ulysses4ever
Copy link
Copy Markdown
Collaborator

I'm fine with $'s for single identifiers. I resolved all my comments from earlier, so to my mind this is ready to go.

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