schaut soweit gut aus und funktioniert, wie es soll.
Mit dem Verschieben der Schriftgrößen-Einstellungen in die Allgemeinen Einstellungen bin ich nicht ganz glücklich, und war aus folgenden Gründen:
- Das Menü der Allgemeinen Einstellungen ist eh schon riesig. Aber das nur nebenbei, denn gewichtiger sind die nächsten Argumente.
- Als Teil der Allgemeinen Einstellungen entsteht für die Standard-Schriftart der Eindruck, dass die Größenanpassung generell für jedwede Verwendung dieser Schriftart gelten würde. Tatsächlich gilt sie ja aber nur für Text-Views (Konsolenausgaben).
- Für alle anderen Kontexte (Kanalwechsel, Wiedergabe von Aufzeichnungen usw.) ist die Einstellung der Schriftgrößen immer in spezifischen Untermenüs angesiedelt. Insofern wäre es für die Konsolenausgabe nur konsequent, dies in gleicher Weise handzuhaben.
Aus den beiden letztgenannten Gründen hatte ich mich auch für ein eigenes Menü entschieden. Außerdem wären wir flexibler, wenn für Text-Views später noch weitere Einstellungen dazukommen sollten.
Wenn es aber so bleiben soll, wäre mein Vorschalg, in der Beschriftung deutlich auf die Konsolenausgabe hinzuweisen, beispielsweise:
- Schriftgröße anpassen - Standard-Schriftart in Textkonsole
- Schriftgröße anpassen - Festbreiten-Schriftart in Textkonsole
Noch einige Gedanken zum Code:
- Da die Festbreiten-Schriftart im OSD-Setup FontFix heißt (was mir nicht bewusst war), wäre es wohl günstig, auch hier diesen Begriff zu nutzen, also fontFixed in fontFix umzubenennen (inkl. der entsprechenden Präfixe).
- Nachdem du in config.c bei SetFonts() für die Standardschrift den Ternary refaktorisiert hast, würde ich für Text-Views ebenfalls einen Teiler von 30 vorschlagen. Damit kam bei meinen Tests automatisch eine Schriftgröße zustande, die eine Anpassung nur selten notwendig macht; anpassbar sollte die Größe der Standard-Schriftart dennoch bleiben. (Leider ist mir das erst nach meinem Post eingefallen.)
Danke nochmals und viele Grüße
Stefan