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

Try out KaTeX with sans-serif font #549

Closed
Entkenntnis opened this issue Sep 3, 2020 · 6 comments · Fixed by #565
Closed

Try out KaTeX with sans-serif font #549

Entkenntnis opened this issue Sep 3, 2020 · 6 comments · Fixed by #565
Assignees
Labels
enhancement New feature or request

Comments

@Entkenntnis
Copy link
Member

Students are sometimes confused by serif font, especially the x.

We could try to default all math formulas to sans-serif and see it this brings an improvement.

@Entkenntnis Entkenntnis added the enhancement New feature or request label Sep 3, 2020
@elbotho
Copy link
Member

elbotho commented Sep 5, 2020

okay, I researched this a bit.
completely switching fonts won't really work with KaTeX.

But KaTeX comes with a sans-serif font that can be activated using using \sf, \mathsf{example} or \textsf{example}. All should result in the same output.

The result… is not great. If it uses sans-serif fonts it's okay, but looks a bit out of place. And then it still uses serifs for some parts. It even displays the same symbols differently sometimes, which is super confusing.

Examples

Pings
@Entkenntnis feel free to play around.
@kathongi take a look!

@elbotho elbotho added this to backlog: decided, waiting (current milestone) in P Responsive Redesign via automation Sep 5, 2020
@elbotho elbotho moved this from backlog: decided, waiting (current milestone) to review / feedback in P Responsive Redesign Sep 5, 2020
@elbotho elbotho self-assigned this Sep 5, 2020
@elbotho elbotho linked a pull request Sep 5, 2020 that will close this issue
@kathongi
Copy link
Member

kathongi commented Sep 7, 2020

I don't understand the problem to be honest. I think it looks quite good with the new font. Not worse than the standard LaTeX font.
I also asked some authors from the academy and got positive feedback. They said it looks more modern and less threatening :D

@elbotho
Copy link
Member

elbotho commented Sep 7, 2020

@kathongi
Mostly things like this put me off, where same variables (a,b) are displayed in two different ways.
You can definitely judge this betten than me … isn't this confusing for learners?
image
(left vs right)

@Entkenntnis
Copy link
Member Author

@elbotho After some investigation all cases of serif fonts are content problems: Your example on the left is using serif fonts explicitly in the latex code (huch?), and the on the page with logarithms, some formulas are not correctly inserted (not using math environment), somehow causing the frontend converter to ignore them (only happen in legacy content).

@kathongi Thank you for the feedback, I do agree that it looks more modern and more like school book style.

So in summary, I would merge the PR into staging.

@elbotho
Copy link
Member

elbotho commented Sep 7, 2020

@Entkenntnis nice, thanks for investigating 👍
It was super easy to find those examples, how should we handle those cases?

(In general im not against merging this :)

Forced serifs: What code is used to archive that, could we re replace to automatically get rid of it in the meantime?

@Entkenntnis
Copy link
Member Author

@elbotho As long as an article is not converted (see https://are-we-edtr-io-yet.serlo.org/), we can assume that project convo will work through these cases.

Otherwise, if we encouter such code in converted content, I would probably talk with the authors. Its mostly not necessary to write such code

P Responsive Redesign automation moved this from review / feedback to Done (merged into staging) Sep 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
P Responsive Redesign
  
Done (merged into staging)
Development

Successfully merging a pull request may close this issue.

3 participants