add katex font KaTeX_Size4-Regular.woff2

[tessus]

fixes #2477

Laurent, I think we should add all *.woff2 fonts.

[tessus]

I forgot to actually mention you @laurent22

Not sure, if you turned off all notifications.

[tessus]

@laurent22 I still think we should add all *.woff2 fonts. They don't take a lot of space. Otherwise we'll create one PR after the other for every katex function someone uses.

[laurent22]

Yes, maybe, but is there any doc about what these extra fonts are for? We had the same four for a long time and it seems it's good enough, so I get the feeling the other ones aren't really needed for us.

[laurent22]

The extra font you've added also needs to be registered in katex.js. We don't load all the assets every time, only those relevant to the active plugins within the note, and so plugins need to tell the app what assets they need.

That makes me think that could actually be automated, but for now we need to do that manually.

[tessus]

Ok, I've registered the font in katex.js.

I've also opened an issue with katex to ask for a list/matrix of what the fonts are used for.
Although I seriously doubt they will reply with an explanation.
They probably tell me that if I wanted to use KaTeX properly, I'd have to load all the fonts because that's why they are there and that it was my fault by not using the entire katex package, but just a part of it.
TBH, I'm sure you would dismiss an error report, if a user had deleted a few files from the Joplin app.

I suspect they are just used for different aspects of KaTeX. e.g. if you use a function x,y,z, font A is used, and so on.

[tessus]

@laurent22 this one should also be ready. btw, I've added comments to make adding other fonts easier in the future. You will only have to uncomment the fonts and done.

[laurent22]

I've also opened an issue with katex to ask for a list/matrix of what the fonts are used for.
Although I seriously doubt they will reply with an explanation.
They probably tell me that if I wanted to use KaTeX properly, I'd have to load all the fonts because that's why they are there and that it was my fault by not using the entire katex package, but just a part of it.

Hmm, yes that's a good point. I've also made some tests and I see that while having the fonts there would increase the package size, it shouldn't affect performances. So I'm ok with including all the fonts as you've prepared in the build script. If you could uncomment the code, we can merge this.

[tessus]

@laurent22 done

[laurent22]

Perfect, thanks!