Vorschlag zu anderem Timeoutverhältnis bei Menus

[Barf]

Bei verschachtelte Menus passiert Folgendes bei Timeout: Bei Timeout verschwindet das letzte Menu, und wird durch sein Vater-Menu ersetzt. Dann startet den Timeoutintervall für den Vater-Menu, und es verschindet danach nach dem Timeoutperiod, und wird durch den Großvatermenu ersezt. Dieses ...

Ich finde dies unbequem und unlogisch. Ich lasse oft ein Menu am Schirm, und lasse es timeout-en. Dann erwarte ich mich natürlich nicht den Vatermenu, sondern eine "reine Schirm", also dass ein time-outed Menu seine Vorgänger "tötet". Meines Wissens funktioniert z.B. Betanova so.

Hier ist ein Patch für ...apps/tuxbox/neutrino/src/gui/widget/menue.cpp. Eine Zeile!

[fernsehass]

ich werds mal einbauen, und berichten, obs funktioniert

geht net, da dann keine timer mehr programmierbar sind

[Nico 77]

Ich würde gerne in einem Menü das timeout komplett abschalten, gibt es da auch eine Idee zu?

[rasc]

mhh, ja im Source die entsprechende Zeile löschen..


(jaja, ich weiss sowas suckt)

[Nico 77]

mhh, ja im Source die entsprechende Zeile löschen..

(jaja, ich weiss sowas suckt)

Wenn ich wüßte welche das wäre....

[fernsehass]

wenn man wüsste, in welchen menü, du das timeout abschalten willst, kann man evtl. helfen

[Nico 77]

In einem eigenen Menü was vom Hauptmenü aus aufgerufen wird um bestimmte Eingaben machen zu können.

[fernsehass]

dann wirkt doch das selbe timeout wie das hauptmenü

[Nico 77]

dann wirkt doch das selbe timeout wie das hauptmenü

Das ist mir schon klar, genau deswegen Frage ich ja wie man den Timeout in einem Menü abschalten kann unabhänig vom Hauptmenü.

Das Hauptmenü soll seine 60sec. Standzeit behalten und dann schliessen nur das eigene Menü nicht.

[fernsehass]

also timeouts kannst ja nur
menü,kanalliste,epg,infobar,filebrowser,Zifferntasten
einstellen. man müsste da noch einen eintrag erstellen für "zusatzmenü". dann sollte das eigentlich kein problem darstellen. ich werd mal die neutrino.cpp guggen, ob ich was finde

PS ich hab mit meinen bescheidenen möglichkeiten nichts gefunden
soweit ich das sehe, wird das in settings.cpp abgearbeitet

const int default_timing[TIMING_SETTING_COUNT] =
{
60,
60,
240,
6,
60,
3
};

const neutrino_locale_t timing_setting_name[TIMING_SETTING_COUNT] =
{
LOCALE_TIMING_MENU,
LOCALE_TIMING_CHANLIST,
LOCALE_TIMING_EPG,
LOCALE_TIMING_INFOBAR,
LOCALE_TIMING_FILEBROWSER,
LOCALE_TIMING_NUMERICZAP
};

[Barf]

Und jetzt, einen (fast) vier Jahren alten Thread ausgegraben!

Spricht etwas dagegen, der in dem ersten Post genannten Patch einzuchecken? Gegen aktuellen CVS:

Code: Alles auswählen

diff -u -r1.147 menue.cpp
--- menue.cpp   1 Jan 2009 23:36:38 -0000       1.147
+++ menue.cpp   1 Feb 2009 12:16:27 -0000
@@ -272,6 +272,7 @@
                                        break;
 
                                case (CRCInput::RC_timeout):
+                                       retval = menu_return::RETURN_EXIT_ALL;
                                        break;
 
                                //close any menue on dbox-key

[Tommy]

Ich fänds gut. Mich nervt das immer (besonders) beim Moviebrowser.
IST (ätzend)
- Film zu Ende
- Filmauswahl (timeout)
- Movieplayer Menü (timeout)
- dbox Menü (timeout)
- TV Bild
SOLL
- Film zu Ende
- Filmauswahl (timeout)
- TV Bild

[rhabarber1848]

Spricht etwas dagegen, der in dem ersten Post genannten Patch einzuchecken?

Ich habe hin und wieder die Situation, wo ich in einem tief verschachtelten
Menü eine Einstellung tätigen möchte, nur dauert es etwas, wenn ich z.B.
meine Optionenliste durchsehen muss, um den passenden Wert zu finden.
Zwischenzeitlich schlägt das Menü-timeout zu, jetzt ist das nicht tragisch,
weil ich nur eine Menüebene tiefer zu gehen brauche. Mit dem von Dir
vorgeschlagenen Patch müsste ich mich wieder durch alle Ebenen hin-
durchhangeln, das kann schon ärgerlich sein. Ich sehe aber dennoch den
Sinn Deines Patches, würde aber vorschlagen, ihn optional zu machen.

[Tommy]

Zusatzvorschlag:
Optional - OK und wenn diese Option gesetzt ist wird auf "Home" genauso gehandelt. Das heißt, wenn man in einem childmenü ist wird auf Home wieder der TV Bildschirm gezeigt. Beim Settingsmenü wäre es dann allerdings erforderlich ein "Settings speichern?" Dialog zwischenzuschalten

[Barf]

Erstmals glaube ich, dass ein Expert auf GUI-Design das jetztige Verhalten als direkt schlecht btw. fehlerhaft bezeichen wurde (aber ich behaupte kein Expert zu sein). "Timeout" bedeutet dass der Benutzer pennt, und dass das System zurück ins "Grundzustand" schnellst mögligst gehen soll, nicht in einem transienten Zwischenzustand.

@rhabarber1848: was du willst ist schnellstmögligst zu dem Zustand zurückzukommen, wo der Timeout eingetroffen war. Hier hat neutrino eine "Lösung" in dem die frühere gewählte Optionen Default sind. Also kommst du zurück (oft am mindestens) mit dBox-Taste, gefolgt von einige "OK"s. Ausserdem solltest du vielleicht eine höhere Timeout setzen.

... Deines Patches würde aber vorschlagen, ihn optional zu machen.

Ich schätze (offensichtlich auch wie Tommy), dass du meinst das Verhältniss konfigurierbar zu machen. Schlechter Vorschlag. Noch ein schwer zu erklärende Option. Schon Namensvergabe scheint problematisch: "Verhalten bei Timeout in verschachtelten Menus: Grundzustand oder Zurückkehr zum Vorgängermenu"...? Igitt...

Ich möchte kurzfristig einiges aufräumen, und habe mich angeboten, entweder ein Patch einchecken, oder es zu lassen. Falls eine Konfigurationsoption etc gewünscht ist, dann nicht von mir, nichteinmal falls ihr mich über die Sinnvollkeit mich überzeugen könnte .

[rhabarber1848]

entweder ein Patch einchecken, oder es zu lassen. Falls eine Konfigurationsoption etc gewünscht ist, dann nicht von mir, nichteinmal falls ihr mich über die Sinnvollkeit mich überzeugen könnte .

Check es ruhig ein, das Konfigurieren-Szenario tritt seltener auf als Tommys
Movieplayer-Szenario. Wenn mich das geänderte Verhalten zu sehr nervt,
überlege ich mir, eine Option dafür zu coden.

[Tommy]

@Barf: Thumbs up

[rhabarber1848]

Patch ist im CVS: http://cvs.tuxbox-cvs.sourceforge.net/l ... 00010.html

[Gaucho316]

Man kann jetzt keine Aufnahme- und Umschalttimer mehr manuell anlegen. Nach der Auswahl des Kanals schließen sich alle Fenster. Ich habe die Vermutung, dass es mit diesem Patch zusammenhängt. In Zeile 363 und 364 von timerlist.cpp wird nämlich

Code: Alles auswählen

g_RCInput->postMsg(CRCInput::RC_timeout, 0); // leave underlying menu also
g_RCInput->postMsg(CRCInput::RC_timeout, 0); // leave underlying menu also

aufgerufen. Oder bin ich da jetzt völlig auf dem Holzweg?

[rhabarber1848]

Ich kann den Fehler bestätigen, Patch ist im CVS erstmal rückgängig gemacht.

[Barf]

Ohmannohmannohmann

Hier hat jemanden, in der Tuxbox Stein- oder Bronzalter, Code geschrieben, die eklatant "Timeout"-Messages zweckentfremdet, um ein Art generische "Du bist fertig"-Mitteilung zu senden. Und dabei ein, ich behaubt fehlerhaftiges, Verhalten vorausgesetzt...

Neutrino ist jetzt was Netscape Navigator 4 am Ende der 90-er Jahren war.

Dank @ Gaucho316 und rhabarber1848.

[dwilx]

Und jetzt? wäre es nicht besser gewesen, statt eines Reverts, sich lieber um eine Lösung dafür zu kümmern?

Man kann jetzt keine Aufnahme- und Umschalttimer mehr manuell anlegen.

@barf

Hier hat jemanden, in der Tuxbox Stein- oder Bronzalter, Code geschrieben, die eklatant "Timeout"-Messages zweckentfremdet, um ein Art generische "Du bist fertig"-Mitteilung zu senden. Und dabei ein, ich behaubt fehlerhaftiges, Verhalten vorausgesetzt...

Neutrino ist jetzt was Netscape Navigator 4 am Ende der 90-er Jahren war.

Man muss zwar etwas überlegen/übersetzen was das bedeutet (nimmt dir aber keiner übel )
Wenn ich dich richtig verstehe, hast du eigentlich recht.

[rhabarber1848]

Und jetzt? wäre es nicht besser gewesen, statt eines Reverts, sich lieber um eine Lösung dafür zu kümmern?

Sicher, aber bis ich C/C++ so gut verstehe, dass ich hier helfen kann, gibt
es viele wütende User, die keine manuellen Timer mehr setzen können
Deshalb habe ich den Patch rückgängig gemacht, mehr kann ich nicht tun.

[dwilx]

Deshalb habe ich den Patch rückgängig gemacht, mehr kann ich nicht tun.

Deine Tat in Ehren, aber ich bin mir sicher, dass barf schon darauf aufmerksam geworden und vlt. das Problem angegangen wäre.

[rhabarber1848]

Deshalb habe ich den Patch rückgängig gemacht, mehr kann ich nicht tun.

Deine Tat in Ehren, aber ich bin mir sicher, dass barf schon darauf aufmerksam geworden und vlt. das Problem angegangen wäre.

Das kann er immer noch tun, aber bis dahin habe ich dafür gesorgt,
dass der CVS-Code funktioniert und die Imagebauer nicht mit
Fehlermeldungen konfrontiert werden, deren Ursache bekannt ist.