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

Use Liberation Seriff as replacemenent for Times New Roman in credits #43

Closed
sheepo99 opened this issue Mar 20, 2019 · 8 comments
Closed
Labels
enhancement New feature or request

Comments

@sheepo99
Copy link

I'm aware this project is all about staying true to the original, but would it be possible to tell the GNU/Linux executable to use Liberation Seriff as a fallback for Times New Roman on the game's credits section? Both fonts are nearly identical to each other, and it would prevent users on GNU/Linux having to install non-free fonts in order to just see the game's credits properly. Future GNU/Linux packages could even require the font as a dependency, for this purpose.

@AJenbo
Copy link
Member

AJenbo commented Mar 21, 2019

Looks ok I think:
image

I think now is a good time to do this, the ports are mature enough that they carry some of there own dependencies (a readme in the case of Linux), expect to see this font included in the next release. Thanks for the suggestion.

@AJenbo AJenbo closed this as completed in bf27555 Mar 21, 2019
@AJenbo AJenbo added the enhancement New feature or request label Mar 23, 2019
@AJenbo
Copy link
Member

AJenbo commented Apr 6, 2019

I think I found a better font:
image
CharisSILB.ttf

@AJenbo
Copy link
Member

AJenbo commented Apr 6, 2019

Here is the original for comparison:
image

@sheepo99
Copy link
Author

sheepo99 commented May 2, 2019

My suggestion is for you to look for a font with better Unicode Support. That font looks more metrically exact to times than Liberation Serif, but it will make it impossible to support, for example, Russian translation.

I would suggest you to test the following ones:

FreeSerif: best unicode support of all Times lookalikes
Linux Libertine: better unicode support than liberation, but still inferior to FreeSerif

@sheepo99
Copy link
Author

sheepo99 commented May 2, 2019

My bad, it does appear Charis, does support Russian and Greek. No Hebrew though:

https://software.sil.org/charis/support/character-set-support/

I would still advise to test libertine and FreeSerif

@AJenbo
Copy link
Member

AJenbo commented May 2, 2019

The font is only used for the credits and one button, all other text is done via the UI or in-game pixel fonts that are limited to a subset of https://en.wikipedia.org/wiki/Windows-1250 and some times even https://en.wikipedia.org/wiki/ASCII (all in-game text) so before that is solved I don't see much point in looking for a credit front that supports Russian. You're however welcome to experiment with various fonts and report if you find one that looks good for the original text/render-style and also supports full Unicode.

@sheepo99
Copy link
Author

sheepo99 commented May 3, 2019

As far as language support goes, I am perfectionist. Russian/Greek/Hebrew support is important because there is a good chance there will be fan translations made for these languages, and keeping the credits section in English would be lack of polish.

Moreover, with mod support coming through, modders might feel the need to use the font as well for their own projects.

Could you let me know how can I replace the font using a compiled version so I can test it? Is it a simple matter of replacing a font file and renaming it?

@AJenbo
Copy link
Member

AJenbo commented May 3, 2019

If you make a build of the current code then just rename the font to CharisSILB.ttf and place it next to the game program. If you are still running 0.3.1 then you will have to place it in this path /usr/share/fonts/truetype/msttcorefonts/Times_New_Roman_Bold.ttf on Windows it would probably be on the same drive as the program is located c:\usr\share\fonts\truetype\msttcorefonts\Times_New_Roman_Bold.ttf. We still have one remaining bug that blocks us from releasing 0.4.0, it shoudn't be too long until it is solved, but at the moment most focus is still given to cleaning up the code.

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
None yet
Development

No branches or pull requests

2 participants