search

ntnu-tex / beamerthementnu

A LaTeX beamer theme for presentations in the NTNU corporate design
https://github.com/ntnu-tex/beamerthementnu/blob/master/ntnu-beamer-documentation.pdf
54 stars 41 forks source link

Mathfont package option #18

Closed kklocker closed 3 years ago

kklocker commented 3 years ago

Fixes #11 .

This seem to work just fine, although it would be "prettier" to not have to pass mathfont=serif as default. I have not managed to do that, but suggestions for improvements are welcome -- I have never done this before :))

kklocker commented 3 years ago

I think at least one of the examples included in the documentation should display the effect of setting mathfont=serif, by adding a sentence on the slides ~"Math mode looks like this: $E = mc^2$"

kellertuer commented 3 years ago

The update looks already quite good, I will check the next days that it works as I hope it does (from reading the code). Thanks for also providing an update to the docs; my only meta-critic would be that the math looks a little boring and something with an integral would be sweet, too. But that is of course just a personal opinion; technically this already looks very good.

Thanks for your work.

kklocker commented 3 years ago

Done :)

kellertuer commented 3 years ago

I think this is great and can be merged. Could you just replace 2021/05/26 v1.0 with 2021/06/28 v1.1 then (just a global search/replace)? This will make it easier when requests/bugs are found or need to be investigated.

kklocker commented 3 years ago

I pushed a tag v1.1 to my local repo, i don't know if this will appear in this repo as well?

kellertuer commented 3 years ago

Yes, that is synced – your local branch (oh you even worked on your master) and this one here. Perfect. Then we can merge.