JGRennison / OpenTTD-patches

OpenTTD - http://www.openttd.org/ - with additional patches
Other
553 stars 127 forks source link

[Bug] Font rendering is ugly on Linux #622

Closed man-of-eel closed 6 months ago

man-of-eel commented 6 months ago

I like to use OpenTTD with these settings:

small_font = Tahoma
medium_font = Tahoma, Bold
large_font = Times New Roman
mono_font = 
small_size = 0
medium_size = 0
large_size = 0
mono_size = 0
small_aa = false
medium_aa = false
large_aa = false
mono_aa = false
On upstream it looks nice, but on JGRPP the letter spacing is all messed up and it's annoying to read. Upstream JGRPP
image image

I'm playing on Arch Linux and for upstream I'm using the Arch package, and for JGRPP I'm using the AUR package.

I thought this might be something to do with the font size since it looks slightly bigger with AA enabled but setting medium_size = 13 in both versions it still looks the same. I think it's the hinting mode being set wrong (maybe full instead of slight) but I couldn't find where that was set in the code. Full hinting has been broken for a few years and it remember it looking fine in Ubuntu 18.04 which is from before the change.

JGRennison commented 6 months ago

I don't have any control of the AUR package. I'd suggest trying with the generic Linux build from here: https://github.com/JGRennison/OpenTTD-patches/releases/tag/jgrpp-0.56.2, or compiling yourself with suitable font rendering libraries installed.

man-of-eel commented 6 months ago

compiling yourself with suitable font rendering libraries installed

That's what the AUR package does, it fetches the source code from here and compiles it (click on View PKGBUILD). FWIW i tried the generic build and it's still there

Happy new year by the way :) I kind of forgot what day it was until i posted it

JGRennison commented 6 months ago

From what I can gather from the issue you linked and the other issues linked from it, there is nothing at all that I can do about this issue. I'd suggest using a different set of fonts.

man-of-eel commented 6 months ago

Turns out this is an upstream bug, not sure how I missed it. Compiling OpenTTD 13.4 there's no bug but latest git it's there. Guess I'll tell em