EPG Vorschau buggy?

[mgck]

Mir ist gerade folgendes aufgefallen:

1. Ich wechsel mit der OK Taste in die Favoritenliste
2. Wähle einen Sender
3. Drücke die rote Taste
4. Wähle eine Sendung aus
5. Drücke nun die ? Taste um mir den EPG Text (Sendungsinfo)anzeigen zu lassen.

So war es eigentlich bisher.
Nun geht Punkt 5 nicht mehr. Nach drücken der ? Taste werden keine Sendungsinformationen angezeigt sonder er sprigt zurück zu Favoritenübersicht.

Bin ich auf einem Kanal und drücke 2x ? dann geht es wie gewohnt.

Ist da irgent etwas verändert worden ?

Dbox2 Philips
Gui: Neutrino
CVS vom 01.07.2007

Bootlog sagt nix dazu.

gruß mgck

[Z80]

selbiges fiel mir auch soeben auf. cvs 04.07.

[ingrid]

Liegt jetzt wohl auf "OK". Vorher war's irgendwie intuitiver, finde ich.

[Z80]

Liegt jetzt wohl auf "OK". Vorher war's irgendwie intuitiver, finde ich.

"OK" gabs zuvor ja zusätzlich zu "?" (funktion war redunant). nur mit "?" gehts nu nimmer.

[deepteam]

Liegt jetzt wohl auf "OK". Vorher war's irgendwie intuitiver, finde ich.

"OK" gabs zuvor ja zusätzlich zu "?" (funktion war redunant). nur mit "?" gehts nu nimmer.

ist mir auch schon aufgefallen, wobei ich die "?" Taste auch intuitiver finde als in der EPG Vorschau auf "OK" zu drücken.

Im Zusammenhang mit dem EPG wurde am 29.06. die Funktion "EPG Infos aktualisieren" ins CVS eingecheckt. Super Funktion, allerdings ist der Text viel zu lang und ragt aus dem Menübereich der EPG Vorschau hinaus (hab die Funktion auf die blaue Taste gelegt). Wenn ich dann das EPG Vorschau Menü wieder verlasse bleibt der hinausragende Text "..ktualisieren" noch im normalen Bild weiter vorhanden.

[Houdini]

jo, die Längenüberprüfung fehlt noch...
Vorschläge für einen kürzeren Text werden gerne entgegengenommen

[mgck]

Wenn die Funktion schon auf eine ander Taste gelegt wird dann bitte auch das Menü updaten.
Rechts oben steht halt noch das ?






gruß mgck

[dbt]

Wäre es so recht?

[dietmarw]

Vorschläge für einen kürzeren Text werden gerne entgegengenommen

EPG Reload

[Gaucho316]

... oder einfach nur "Neu laden" im Deutschen und "Reload" im Englischen. "EPG" muss meiner Meinung nach nicht vorkommen, da wir ja wissen, dass wir im EPG sind.

[ingrid]

@Houdini:

eventlist.cpp / Zeile 421ff

Code: Alles auswählen

		else if ( msg==CRCInput::RC_left || msg==CRCInput::RC_red || msg==CRCInput::RC_help)
		{
			loop= false;
		}

		else if (msg==CRCInput::RC_help || msg==CRCInput::RC_right || msg==CRCInput::RC_ok)
		{
			if ( evtlist[selected].eventID != 0 )

Ist das wirklich korrekt? Oder müsste es nicht so heißen?

Code: Alles auswählen

		else if ( msg==CRCInput::RC_left || msg==CRCInput::RC_red)
		{
			loop= false;
		}

		else if (msg==CRCInput::RC_help || msg==CRCInput::RC_right || msg==CRCInput::RC_ok)
		{
			if ( evtlist[selected].eventID != 0 )

Ansonsten wird msg==CRCInput::RC_help in der zweiten Abfrage ja nie angesprungen, oder?

Nur so 'ne Vermutung. Wenn das so sein soll, dann bastel ich mir 'nen Patch für meine Hilfe-Taste in der EPG Liste. Wenn's 'n Versehen ist, spare ich mir den lokalen Patch...

Oder gibt's 'nen wichtigen Grund, warum die Help-Taste dort "rausgenommen" wurde, also die if-Abfragen SO sind, wie sie JETZT im CVS sind?

[ingrid]

Falls jemand Interesse hat:

Code: Alles auswählen

Index: apps/tuxbox/neutrino/src/gui/eventlist.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/gui/eventlist.cpp,v
retrieving revision 1.106
diff -U3 -r1.106 eventlist.cpp
--- a/apps/tuxbox/neutrino/src/gui/eventlist.cpp	29 Jun 2007 20:27:54 -0000	1.106
+++ b/apps/tuxbox/neutrino/src/gui/eventlist.cpp	6 Jul 2007 14:18:16 -0000
@@ -418,7 +418,7 @@
 			loop=false;
 		}
 
-		else if ( msg==CRCInput::RC_left || msg==CRCInput::RC_red || msg==CRCInput::RC_help)
+		else if ( msg==CRCInput::RC_left || msg==CRCInput::RC_red)
 		{
 			loop= false;
 		}

[dbt]

Ich hatte das mit dem Beispiel oben so gelöst wobei auch die Tasten < und > ebenfalls greifen. Irgendwie ziehmlich "universal". Wird schon irgendwie 'ne Variante geben

Code: Alles auswählen

Index: eventlist.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/gui/eventlist.cpp,v
retrieving revision 1.106
diff -u -r1.106 eventlist.cpp
--- eventlist.cpp	29 Jun 2007 20:27:54 -0000	1.106
+++ eventlist.cpp	6 Jul 2007 14:26:15 -0000
@@ -560,7 +560,8 @@
 	liststart = (selected/listmaxshow)*listmaxshow;
 
 	if (evtlist[0].eventID != 0)
-		frameBuffer->paintIcon(NEUTRINO_ICON_BUTTON_HELP, x+ width- 30, y+ 5 );
+		frameBuffer->paintIcon(NEUTRINO_ICON_BUTTON_OKAY, x+ width- 60, y+ 5 );
+		frameBuffer->paintIcon(NEUTRINO_ICON_BUTTON_HOME, x+ width- 30, y+ 5 );
 
 	for(unsigned int count=0;count<listmaxshow;count++)
 	{
@@ -969,4 +970,3 @@
 	res = searchMenu.exec(NULL,"");
 	return(res);
 }
-

[ingrid]

Ich hatte das mit dem Beispiel oben so gelöst wobei auch die Tasten < und > ebenfalls greifen. Irgendwie ziehmlich "universal". Wird schon irgendwie 'ne Variante geben

Nur "Rechts" geht, "Links" und "Rot" springt raus. (siehe eventlist.cpp)

Also:
Vorher:
Rechts/OK/Help --> Auswahl
Links/Rot/Home --> Exit

Jetzt im CVS:
Rechts/OK --> Auswahl
Links/Rot/Home/Help --> Exit

Mal sehen, was Houdini sagt, ob's Absicht ist. Wenn's 'n Versehen ist, kann man sich das Austauschen der Icons erstmal sparen...

[Houdini]

hmm, ich hatte es in einer ersten Version mal auf die ? Taste gelegt, es dann aber auf eine Farbtaste konfigurierbar gemacht, das ist dann wohl verschütt gegangen

[PauleFoul]

Also bei mir ist der Text für die neu Funktion "EPG neu laden" auch
völlig daneben. Ist mir auch net klar wie da selbst der verkürzte Text
angezeigt werden soll!?



Warum kann man dann nicht einfach eine Zeile unter die anderen Buttons
gehen? Ich habe die Funktion im übrigen auf die Dbox-Taste gelegt...


Gruß
____Paule

[mgck]

Ich habe es mir jetzt so eingerichtet, da ja beide tasten funktionieren.



gruß mgck

[Houdini]

@Paule
momentan ist nur Platz für 4 Buttons
da musst du halt nen anderen rausschmeissen

[PauleFoul]

@Paule
momentan ist nur Platz für 4 Buttons
da musst du halt nen anderen rausschmeissen

Leider könnte ich nur auf "Sortieren" verzichten und den bekommt
man net weg. Dann lieber keinen Text einblenden wenn 4 Buttons
da sind. Das sollte man doch abfragen können, oder??


Gruß
____Paule

[Houdini]

klar kann man, aber:
Einstellungen-Tasten-Sortierreihenfolge ändern auf keine Taste und speichern

[eviltrooper]

gibts jetzt ne lösung das man es wieder mit ? aufrufen kann???

[Gaucho316]

http://cvs.tuxbox.org/lists/tuxbox-cvs- ... 00008.html

[Gorcon]

gibts jetzt ne lösung das man es wieder mit ? aufrufen kann???

Würde mich auch interessieren.

Gruß Gorcon

[ingrid]

Würde mich auch interessieren.

Ist schon im CVS...

houdini 07/07/06 23:07:23

Modified: tuxbox/neutrino/src/gui eventlist.cpp
Log:
restored old eventlist behaviour for ? key

Revision Changes Path
1.107 +2 -2 apps/tuxbox/neutrino/src/gui/eventlist.cpp

[Gaucho316]

Was meint ihr, wozu ich den Link gepostet habe?