[Patch] Schriftgrößenfaktor

[GetAway]

Ich habe mal den Fontrenderer erweitert, bzw. die Änderungen aus Neutrino-HD portiert
und dem Schriftmenü ebenfalls die Funktion Schriftgrössenfaktor gegönnt. Das steht nicht
nur Neutrino gut, sondern auch meinem LCD.

Link entfernt, da Patch in Git

[Gaucho316]

Warum hast du eigentlich den Aufruf der Methode init() in der Klasse CStringInput vom Konstruktor in die Methode paint() verschoben? Und warum hast das nur in einem der beiden Konstruktoren geändert? Wenn das da auch raus muss, kann ich heute Abend gerne einen kleinen Patch dafür schreiben. Ich habe nämlich auch zwei kleine Optimierungen für die Änderungen in COsdSetup im Kopf. Die können wir ja dann ins Git nachschieben.

[GetAway]

Hi,
weil sonst nach Änderung Schriftgrössenfaktors, bei CStringinput, die Buchstaben oben aus der Box geragt hätten.
Ist mir so nebenbei aufgefallen, aber da gibts noch andere Stellen, die man anpassen muss. Fällt natürlich nur auf,
wenn man es mit dem Zoom übertreibt. Mach die Schrift mal besonders hoch, dann siehst du einiges.

Ich glaube das 2. init() kann ganz weg. Wird nimmer gebraucht an der Stelle. Kann es aber jetzt nicht testen.

Achja, im sectionsd wurde wieder was geändert. Ist doch deine Baustelle

[doc]

Wenn man solche Veränderungen im Commitlog begründet und darlegt kann man das auch später noch nachvollziehen.

[Gaucho316]

Warum hast du eigentlich den Aufruf der Methode init() in der Klasse CStringInput vom Konstruktor in die Methode paint() verschoben? Und warum hast das nur in einem der beiden Konstruktoren geändert? Wenn das da auch raus muss, kann ich heute Abend gerne einen kleinen Patch dafür schreiben. Ich habe nämlich auch zwei kleine Optimierungen für die Änderungen in COsdSetup im Kopf. Die können wir ja dann ins Git nachschieben.

Ich glaube das 2. init() kann ganz weg. Wird nimmer gebraucht an der Stelle.

follow-up to commit 07fcbdc
Link entfernt, da Patch in Git

Achja, im sectionsd wurde wieder was geändert. Ist doch deine Baustelle

Ich meine, alles portiert zu haben, was auch für Neutrino brauchbar ist.

[GetAway]

Im Fontrenderer ist mir noch etwas durchgegangen.
0001-fontrenderer.cpp-add-forgotten-fix-from-coolstream-p.patch

[doc]

Hmm, und und was ist jetzt der "forgotten fix" ...
Schlechte Headline für einen Commit.

[Gaucho316]

Ist mir so nebenbei aufgefallen, aber da gibts noch andere Stellen, die man anpassen muss.

Und schon habe ich etwas gefunden.

CStringInput: move init of 'selected' into constructors
Link entfernt, da Patch in Git

CMenuOption*Chooser: get current font height in paint()
Link entfernt, da Patch in Git

[Mourice]

Hmm, und und was ist jetzt der "forgotten fix" ...
Schlechte Headline für einen Commit.

Hi,

mich würde eher interessieren, wie es zu einem "Checkin ins Git" kommen kann ohne "Real-Name".

Aber wahrscheinlich gelten die Regeln nicht für alle.

MfG

[dbt]

http://www.tuxbox-cvs.sourceforge.net/f ... 17#p385317

[Gaucho316]

Und noch einer. Ich hoffe, dass der nichts kaputt macht.

Neutrino: reinit infobar, EPG viewer and event list after changing font size
Link entfernt, da Patch in Git


Dieser Patch und der vorherige (CMenuOption*Chooser: get current font height in paint()) sind aber nur Tropfen auf dem heißen Stein. Nach einer Änderung der Schriftgröße oder des Schriftgrößenfaktors ist ein Neustart von Neutrino angeraten. Die Größenberechnungen passieren nämlich vielfach in den Konstruktoren. Und ich habe keine Lust, das überall umzustellen.

[Gaucho316]

Nach einer Änderung des OSD-Bereichs gilt übrigens dasselbe. Auch hier ist ein Neustart von Neutrino angeraten, weil viele Größen- und Positionsberechnungen in Konstruktoren stattfinden. Der folgende Patch löst zumindest eine Neuberechnung des EPG-Info-Fensters und der EPG-Liste aus, weil das dort Recht einfach anzustoßen ist. Für eine teilweise Vorschau, was man so verbrochen hat, reicht's aber.

Neutrino: reinit EPG viewer and event list after changing OSD region
Link entfernt, da Patch in Git


Nebenbei sind mir jetzt auch ein paar "rauslaufende" Texte aufgefallen, wenn man am Schriftgrößenfaktor rumdreht. Dafür gibt es demnächst auch noch einen Patch.

[Gaucho316]

GetAway, deine kleine Änderung in CStringInput hat die restlichen Klassen in stringinput.cpp kaputt gemacht. Eingabefenster vom Typ CStringInputSMS hatten die falsche Größe und beim Öffnen der Jugendschutzabfrage flog mir Neutrino komplett um die Ohren. Guck bitte nächstes Mal genauer, welche Auswirkungen eine vermeintlich kleine Änderung hat.

stringinput.cpp: fix paint in input classes
Link entfernt, da Patch in Git

[GetAway]

Das sollte natürlich nicht passieren. Danke für Fixen. Das sollte dann aber auch zügig ins Git.

[dbt]

Das sollte dann aber auch zügig ins Git.

Ich hab jetzt etwas den Überblick verloren, welche Patche stehen jetzt noch aus?

[Gaucho316]

Alle, die in diesem Thread verlinkt sind. Und der hier und der hier. Das sind insgesamt 9 Patches.

[Gaucho316]

Nebenbei sind mir jetzt auch ein paar "rauslaufende" Texte aufgefallen, wenn man am Schriftgrößenfaktor rumdreht. Dafür gibt es demnächst auch noch einen Patch.

Hier nun die versprochenen Patches.

Neutrino: fix overflowing text in infobar
Link entfernt, da Patch in Git

Neutrino: fix calculating max button width in button bar
Link entfernt, da Patch in Git

Guckt mal, ob alle Texte in den Buttonleisten noch vollständig sind, wenn der Standard-Schriftgrößenfaktor eingestellt ist. Und bei größeren Werten sollte nun nichts mehr aus den Buttonleisten rauslaufen. Buttonleisten gibt es in:

• Audioplayer
• Internetradio
• Bookmarkliste im Movieplayermenü
• Bouquetliste
• Kanalliste
• Vorschau Komplettübersicht (EPG Plus)
• EPG-Infofenster
• EPG-Liste
• Dateibrowser
• Image Informationen
• Infobar (TV, Radio, Movieplayer)
• Motor-Setup
• Bildbetrachter
• Technische Informationen
• Timerliste
• UPnP Browser
• Bouquet-Verwaltung (alle 3 möglichen Fenster)
• SMS-Eingabefeld (z.B. im Film-Informationsmenü des Moviebrowsers)

Ich kann Internetradio, Motor-Setup und UPnP Browser nicht selbst testen. Und zwei Texte musste ich kürzen. In der EPG-Liste heißt "EPG aktualisieren" nun "Aktualisieren" und in der Movieplayer-Infobar "Starte Plugin" nur noch "Plugin".

Der zweite Patch "beißt" sich übrigens in channellist.cpp mit GetAways MiniTV-Patch. Aber da ich mir den ja demnächst so oder so mal genauer ansehen und ein klein wenig überarbeiten will, kann ich das ja gleich mit anpassen.

[dbt]

...Das sollte dann aber auch zügig ins Git.

Habe jetzt alles eingecheckt. (Falls ich jetzt nichts übersehen habe )
http://sourceforge.net/p/tuxbox-cvs/app ... log/?path=

[doc]

Habe jetzt alles eingecheckt. (Falls ich jetzt nichts übersehen habe )

Ein Forum ist eben nichts für so einen Workflow und Mail scheint schwierig zu bedienen zu sein.

[Gaucho316]

Und da ist er wieder - der Meckerfritze vom Dienst ...