search

latextemplates / scientific-thesis-template

LaTeX template for Master, Bachelor, Diploma, and Student Theses
http://latextemplates.github.io/scientific-thesis-template/
BSD Zero Clause License
358 stars 120 forks source link

Computer Modern text in `\mathrm` #112

Open datMaffin opened 5 years ago

datMaffin commented 5 years ago

Screenshot:

Screenshot 2019-11-07 at 15 24 21

I resolved the problem by using the second solution of the Stack Overflow answer https://tex.stackexchange.com/a/394137:

  1. Add [NFSSFamily=ntxtlf] to \setmainfont.
  2. Remove the old solution (as described in the same Stack Overflow question https://tex.stackexchange.com/a/394139/9075):
  3. Move \RequirePackage{newtxmath} below Fontspec commands.

Except for the now correct font in \mathrm I was only able to spot a difference in the rendering of the equals sign. Everything else looks exactly the same!

Since I am not sure about the implications of this change, I am not creating a pull request.

Latex Setup: MacTex / TexLive 2019 with the latest updates

koppor commented 5 years ago

Just go ahead with the PR. It is more easy to play around if a solution is proposed. I would just quickly check it (as I am currently applying fixes and not fully developing the template).

Side note: I am working on https://github.com/latextemplates/generator-latex-template to enable better tests for changes.

koppor commented 2 years ago

Is this fixed by https://github.com/latextemplates/scientific-thesis-template/pull/113?

datMaffin commented 2 years ago

Yes #113 was the original fix.

It will also still work with #142 (using unicode-math for lualatex).