EPG Titel im LCD

[Gaucho316]

Wenn ich mir im Movieplayer eine Aufnahme ansehe und sich das aktuelle Event im EPG des zuletzt gesehenen Fernsehsenders ändert, also eine neue Sendung anfängt, wird das LCD aktualisiert, so dass nicht mehr der Name der Aufnahmedatei sondern der Name des zuletzt gesehenen Fernsehsenders erscheint.

Das scheint jetzt gefixt zu sein. Zumindest konnte ich das die letzten Tage mit dem JtG-Image vom 22.12. nicht mehr beobachten. Allerdings tritt der Fehler jetzt (oder vermutlich immernoch) im Moviebrowser auf. Dort ist dann eben nicht die normale Anzeige im LCD zu sehen sondern Sendername und ggf. Sendungstitel des zuletzt gesehenen Senders.

[rhabarber1848]

Der Silvester-Patch http://cvs.tuxbox-cvs.sourceforge.net/l ... 00171.html

Improve channel name and EPG display in LCD by centering the text and by more intelligent word splitting.

gefällt mir gut, nur möchte ich nochmals vorschlagen, die Möglichkeit vorzusehen,
den Sendernamen zugunsten des Sendungsnamens zu beschränken, wenn beides
nicht komplett auf das Display passt.

Hier das LCD-Display in der Version von seife:


und hier aus einem darkside-Image
(bitte verzeiht, ich habe kein besseres Beispiel gefunden):

Das Zentrieren der Texte, so wie seife es jetzt macht, ist für mich
in Ordnung, die Kombination beider Anzeigen - Sendername einzeilig,
alles zentriert - wäre für mich der Idealfall.

Das

sieht für mich halt besser aus, als das

[GetAway]

Dito.

[seife]

http://cvs.tuxbox-cvs.sourceforge.net/c ... 64&r2=1.65
Irgendjemand muss das halt noch konfigurierbar machen. Die GUI-Sachen sind nicht so meins

Das mit dem Zentrieren fand ich halt elegant, wobei ich momentan überlege die Zentrierung auszumachen, wenn das "smart splitting" für den EPG-Text fehlschlägt, weil ich sowas (Achtung, ASCII-art ) nicht sooo toll finde:

Code: Alles auswählen

+-------------------+
|Karl Moiks Chiemgau|
|  er Volkstheater  |
+-------------------+

Die "Nottrennung" würde so evtl. weniger auffallen:

Code: Alles auswählen

+-------------------+
|Karl Moiks Chiemgau|
|er Volkstheater    |
+-------------------+

Andererseits ist es den extra-Aufwand vermutlich nicht wert.

[rhabarber1848]

Die "Nottrennung" würde so evtl. weniger auffallen:

Ich denke schon, dass es einen Platz auf Deiner To-Do-Liste wert wäre,
aber recht weit unten. Ich teste Deinen neuen Patch gleich mal und schaue,
ob ich die GUI-Config dazu bauen kann.

[rhabarber1848]

Irgendjemand muss das halt noch konfigurierbar machen. Die GUI-Sachen sind nicht so meins

Ungetestet: neutrino_lcd_nameshort.diff

PS: Mittlerweile getestet, sieht gut aus.
@seife: Danke!

[GetAway]

Bei mir geht's so nicht. Die jeweiligen anderen Dateien werden ebenfalls angemeckert.

Code: Alles auswählen

checking locals.h
--- /tmp/log1	2009-01-03 02:13:36.000000000 +0100
+++ /tmp/log2	2009-01-03 02:13:36.000000000 +0100
@@ -543,9 +543,9 @@
  LOCALE_LCDMENU_EPG_NAME,
  LOCALE_LCDMENU_EPG_NAME_SEPLINE_TITLE,
  LOCALE_LCDMENU_EPG_NAME_TITLE,
- LOCALE_LCDMENU_EPG_NAMESHORT_SEPLINE_TITLE,
- LOCALE_LCDMENU_EPG_NAMESHORT_TITLE,
  LOCALE_LCDMENU_EPG_TITLE,
+ LOCALE_LCDMENU_EPG_NAMESHORT_TITLE,
+ LOCALE_LCDMENU_EPG_NAMESHORT_SEPLINE_TITLE,
  LOCALE_LCDMENU_HEAD,
  LOCALE_LCDMENU_INVERSE,
  LOCALE_LCDMENU_LCDCONTROLER,
ERROR: locals.h not ordered or contains empty lines

[Houdini]

da kann jemand das ABC nicht.. *duck*

[Striper]

Hehe, setzen 6!

[rhabarber1848]

Wo ist das Problem? make yadd-neutrino läuft problemlos durch und der Code funktioniert.
PS: make flash-neutrino-squashfs-all läuft auch durch

Müssen die Einträge in locals.h alphabetisch sortiert sein?

[Striper]

Müssen die Einträge in locals.h alphabetisch sortiert sein?

Ja.

[rhabarber1848]

neutrino_lcd_nameshort.diff

Patch aktualisiert, wieder etwas dazugelernt, danke!

Wann tritt die Fehlermeldung eigentlich auf?
Ich kompiliere mit --enable-maintainer-mode und der Fehler trat nicht auf,
obwohl in apps/tuxbox/neutrino/data/locale/Makefile.am das check-target
nur aktiv ist, wenn MAINTAINER_MODE aktiv ist.

Ich konnte den Fehler nachvollziehen, nachdem ich in
apps/tuxbox/neutrino/data/locale "make check" aufgerufen habe.

[-=HSKc=-Robby]

@seife:
Da Riker gestern mal nen neuen JtG-Snapshot erstellt hat, bin ich nun auch in den Genuss des zentrierten EPG-Titels im LCD gekommen. Allerdings is mir ein Bug aufgefallen. Mit der Zeit (beim zappen) bildet sich am linken Rand ein mehr oder weniger langer senkrechter Strich von einem Pixel Breite. Geht man einmal per "D-Box" ins Menü ist er wieder verschwunden. Genauso verschwindet er bei der "OK"-Taste oder einer Feature-Taste.
Kann es sein, dass beim zappen die Spalte ganz links nicht richtig gelöscht wird, falls sie genutzt wurde?

Hab mal nen Bild angehängt. Ist zwar sehr unscharf, habs aber leider nicht besser hinbekommen...

[seife]

Ja, das ist so, ich weiss aber noch nicht genau, warum.
Irgendwas war da, dass das LCD mit den Koordinaten (-1, -1) anfängt oder so ähnlich. Müsste sich mal jemand anschauen.

[Gaucho316]

Ich zitiere mich mal selbst.

Allerdings tritt der Fehler jetzt (oder vermutlich immernoch) im Moviebrowser auf. Dort ist dann eben nicht die normale Anzeige im LCD zu sehen sondern Sendername und ggf. Sendungstitel des zuletzt gesehenen Senders.

Ich habe inzwischen herausgefunden, wann das auftritt. Hat man beim Abspielstart eine Tonspur auszuwählen bzw. drückt die Tonwahltaste (grüne Taste) während des Abspielens, erscheint nach Auswahl einer Tonspur die falsche LCD-Anzeige, also Sendername und Sendungstitel des zuletzt gesehenen Senders statt dem Namen der Aufnahmedatei. Ich möchte darum bitten, dass sich das nochmal jemand ansieht.

[Striper]

@Gaucho316

Müsste bereits gefixt sein: http://cvs.tuxbox-cvs.sourceforge.net/l ... 00146.html

[Gaucho316]

Ich denke nicht. Das war doch ein ganz anderes Problem.

[Striper]

Ich denke nicht. Das war doch ein ganz anderes Problem.

Sorry, hatte dich mit -=HSKc=-Robby verwechselt.

[rhabarber1848]

@seife:

dbox-Taste/Einstellungen/LC-Display
Power aus/ein

hat keine Funktion mehr, das Display ist immer an.
Könnte es sein, dass hier mit last_toggle_state_power etwas durcheinander geraten ist?

[seife]

Keine Ahnung, muss ich mal schauen.

[seife]

Ja, da habe ich beim Übersetzen der alten Logik ein "&" mit einem "|" verwechselt

probier' bitte mal

Code: Alles auswählen

--- driver/lcdd.cpp     17 Jan 2009 12:40:55 -0000      1.66
+++ driver/lcdd.cpp     20 Jan 2009 08:22:48 -0000
@@ -269,14 +269,14 @@ void CLCD::setlcdparameter(void)
        int dim_time = atoi(g_settings.lcd_setting_dim_time);
        int dim_brightness = atoi(g_settings.lcd_setting_dim_brightness);
        bool timeouted = (dim_time > 0) && (timeout_cnt == 0);
-       int brightness, power = last_toggle_state_power;
+       int brightness, power = 0;

        if (timeouted)
                brightness = dim_brightness;
        else
                brightness = g_settings.lcd_setting[SNeutrinoSettings::LCD_BRIGHTNESS];

-       if (! timeouted || (dim_brightness > 0))
+       if (last_toggle_state_power && (!timeouted || dim_brightness > 0))
                power = 1;

        if (mode == MODE_STANDBY)

(copy'n paste, also musst du es von Hand anwenden)

Damit sollte es wieder wie vorher tun. Naja. Vielleicht.
(...ich suche immer noch nach einem Ausdruck, der diese Logik leichter verständlich beschreibt, wenn ich die boole'schen Ausdrpcke in setlcdmodeparameter() durchdenke, wird mir immer ganz schwindelig...)

[GetAway]

Ich hatte die Klammern so gesetzt, damit funktioniert's auch

Code: Alles auswählen

+       if (last_toggle_state_power && (!timeouted || (dim_brightness > 0)))

[rhabarber1848]

probier' bitte mal

Der Patch, der jetzt auch im CVS ist, funktioniert.

[seife]

Ja. Ich lasse gerne überflüssige Klammern (das sind die, die die Lesbarkeit nicht erhöhen) weg.
Welche die Lesbarkeit erhöhen, ist natürlich immer Ansichtssache...

[Gaucho316]

Hat man beim Abspielstart eine Tonspur auszuwählen bzw. drückt die Tonwahltaste (grüne Taste) während des Abspielens, erscheint nach Auswahl einer Tonspur die falsche LCD-Anzeige, also Sendername und Sendungstitel des zuletzt gesehenen Senders statt dem Namen der Aufnahmedatei.

Ich glaube, dass ich die Lösung für dieses Problem gefunden habe. Dieser Patch ist aber wie immer ungetestet.

Code: Alles auswählen

--- movieplayer.cpp.ORIG	2009-01-24 16:55:53.000000000 +0100
+++ movieplayer.cpp	2009-01-24 16:58:00.000000000 +0100
@@ -3540,6 +3540,7 @@
             APIDSelector.exec(NULL, ""); // otherwise use Dialog
             delete APIDChanger;
 			g_showaudioselectdialog = false;
+			update_lcd = true;
 
 			if(g_show_movieviewer)
 				showMovieViewer();