shithub: choc

Info  •  Files  •  Log  •  Branches

textscreen: Remove screen size-based font selection.

This was removed on Windows a while back in favour of a more
conservative approach that examined the system DPI settings, but the
logic still remained for other systems and it really isn't a good
idea. Remove the logic that chooses a large font on large monitors,
and add a TODO comment to add proper Linux font selection in the
future, based on the Gtk+ HiDPI setting.

--- a/textscreen/txt_sdl.c

+++ b/textscreen/txt_sdl.c

@@ -218,12 +218,12 @@

     {

         font = &large_font;

     }

-#else

-    else if (info->current_w >= 1920 && info->current_h >= 1080)

-    {

-        font = &large_font;

-    }

 #endif

+    // TODO: Detect high DPI on Linux by inquiring about Gtk+ scale

+    // settings. This looks like it should just be a case of shelling

+    // out to invoke the 'gsettings' command, eg.

+    //   gsettings get org.gnome.desktop.interface text-scaling-factor

+    // and using large_font if the result is >= 2.

     else

     {

         font = &main_font;