Menüs aufgeräumt

[dbt]

thx, sollte wieder passen

[dwilx]

Deshalb mein Vorschlag: Die Code-Restrukturierung fortsetzen, damit spätere sichtbare Menüänderungen einfacher umzusetzen sind. Letzteres noch mal etwas zurückstellen, damit auch andere User noch ihre Meinung zu Protokoll geben können. Vielleicht sollte man das noch im JTG-Board ankündigen, da dort vermutlich User sind, die hier nicht mitlesen.

http://www.jackthegrabber.de/viewtopic. ... 24&start=0

[mb405]

ich hab die cvs änderungen gemacht, und mir hauts die movieplayer_setup.cpp um die ohren

Code: Alles auswählen

movieplayer_setup.cpp: In member function `void CMoviePlayerSetup::showMoviePlayerSelectPlugin()':
movieplayer_setup.cpp:244: error: `g_PluginList' undeclared (first use this function)
movieplayer_setup.cpp:244: error: (Each undeclared identifier is reported only once for each function it appears in.)
movieplayer_setup.cpp:246: error: `CPlugins' has not been declared
movieplayer_setup.cpp:246: error: `P_TYPE_TOOL' undeclared (first use this function)
make[5]: *** [movieplayer_setup.o] Fehler 1

ich hab keine ahnung wo g_Pluginlist declariert wird. der selbe code in der neutrino.cpp geht.

[dbt]

Müsste in global.h drin stehen, da wurde aber nichts geändert, aber das sieht so aus, also ob irgendwas nicht includiert ist. Ich hab das zichmal hin und her gebaut, ohne Probleme. Bist du sicher, dass du alle Änderungen auch an den makefiles usw. drin hast?

[mb405]

ok danke dir
der eintrag mit den plugins hat tatsächlich gefehlt in global.h
tsts
hab ich wohl änderung 1.52 auf 1.53 verpasst.

[Gaucho316]

Sind nicht eigentlich die folgenden Locales auch überflüssig geworden:

• audioplayeresoundsettings.general
• audioplayerpicsettings.general
• pictureviewer.esound.settings.general

[dbt]

Sind nicht eigentlich die folgenden Locales auch überflüssig geworden:

• audioplayeresoundsettings.general
• audioplayerpicsettings.general
• pictureviewer.esound.settings.general

Glaube schon.

[dbt]

Der nächste größere Batzen wird die Kanalsuche.

Hier die Kanalsuche, erstmal nur um zu sehen, dass diesmal auch die Einstellungen angenommen werden. Für den Benutzer ist alles beim Alten.
neutrino-menue_cleanup_scansetup_diff-2009-10-15-16-08-15.patch

[PauleFoul]

Werden die Änderungen die funktionieren auch mal eingecheckt oder gibt das dann den "großen Wurf"?

[GetAway]

Ja, funktioniert, ist nur beim ersten Aufruf verzögert.
Übrigens ist Dein exitRun-code mit reingerutscht.

@Paule
Ist doch alles drin, außer das von heute.

[dbt]

Werden die Änderungen die funktionieren auch mal eingecheckt oder gibt das dann den "großen Wurf"?

Wie getaway schon sagte, sind die letzten Sachen schon drin. Alles auf einmal, wäre einfach zu viel. Das würde wärend man dran arbeitet Konflikte hageln. Das ist nicht so toll wenn man dann immer hinterher laufen muss, um laufende Änderungen nicht zu verpassen. So gehts noch halbwegs verträglich zu. Ausserdem kann man die übernommenen Änderunge erst mal sacken lassen, um falls notwendig etwas Nachlese machen zu können.

Ja, funktioniert, ist nur beim ersten Aufruf verzögert.
...

Vermutlich weil erst das Menü beim Aufruf aus dem Servicemenü erzeugt wird und nicht wie bisher schon beim Neutrinostart. Es wird also dafür noch kein Speicher verbraten.

[mb405]

ist doch alles eingecheckt.
nur die scaneinstellungen noch nicht

[PauleFoul]

ist doch alles eingecheckt.
nur die scaneinstellungen noch nicht

OK, danke für die Info. War mir irgenwie nicht aufgefallen...

[rhabarber1848]

War mir irgenwie nicht aufgefallen...

Kennst Du schon http://news.gmane.org/gmane.comp.video.tuxbox.scm ?
Damit bist Du immer auf dem Laufenden.

[dbt]

@dbt

Ich habe mal einen alten Patch von Dir genommen und daraus, für das aktuelle CVS,
die Kanalsuche ausgelagert. Leider werden weder die Werte initialisiert, noch lassen sie
sich speichern. Das war aber schon bei dem alten Patch so. Ich habe den Fehler nicht
gefunden. Könntest Du mal einen Blick drauf werfen und es richten, dann kann es getestet
werden und ins CVS. Danke.

Das müsste es jetzt sein, bitte testen
neutrino-menue_cleanup_scansetup_diff-2009-10-15-22-40-15.patch

[rhabarber1848]

Ist movieplayer_setup.cpp nicht doppelt?
Patchvorschlag:

Code: Alles auswählen

--- apps/tuxbox/neutrino/src/gui/Makefile.am     2009-10-15 07:20:04.000000000 +0200
+++ apps/tuxbox/neutrino/src/gui/Makefile.am  2009-10-15 23:05:25.000000000 +0200
@@ -76,7 +76,7 @@
 libneutrino_gui_a_SOURCES += movieplayer2.cpp
 else
 libneutrino_gui_a_SOURCES += \
-       movieviewer.cpp movieplayer.cpp movieplayer_setup.cpp
+       movieviewer.cpp movieplayer.cpp
 endif
 endif

===========================================================================
aus gui/neutrino_menu.cpp

Code: Alles auswählen

#if defined(ENABLE_AUDIOPLAYER) || defined(ENABLE_PICTUREVIEWER) || defined(ENABLE_ESD) || defined(ENABLE_MOVIEPLAYER)
        personalize->addItem(mainSettings, LOCALE_MEDIAPLAYERSETTINGS_GENERAL, true, NULL, new CMediaPlayerSetup, NULL, CRCInput::RC_blue, NEUTRINO_ICON_BUTTON_BLUE, false, g_settings.personalize_mediaplayer);
#endif

Der Code ist in Ordnung, "new CMediaPlayerSetup" wird nur aufgerufen,
wenn eines der Module aktiv ist. In src/gui/Makefile.am ist
mediaplayer_setup.cpp immer Teil von libneutrino_gui_a_SOURCES.
Ungetester Patch, der auch esd_setup.cpp richtig einordnet:

Code: Alles auswählen

--- apps/tuxbox/neutrino/src/gui/Makefile.am     2009-10-15 07:20:04.000000000 +0200         
+++ apps/tuxbox/neutrino/src/gui/Makefile.am  2009-10-15 23:09:50.000000000 +0200                 
@@ -27,13 +27,11 @@                                                                                 
        epg_menu.cpp \                                                                              
        epgplus.cpp \                                                                               
        epgview.cpp \
-       esd_setup.cpp \
        eventlist.cpp \
        favorites.cpp \
        filebrowser.cpp \
        imageinfo.cpp \
        infoviewer.cpp \
-       mediaplayer_setup.cpp \
        neutrino_menu.cpp \
        nfs.cpp \
        personalize.cpp \
@@ -57,21 +55,21 @@
 endif

 if ENABLE_ESD
-libneutrino_gui_a_SOURCES += esound.cpp
+libneutrino_gui_a_SOURCES += esound.cpp esd_setup.cpp mediaplayer_setup.cpp
 endif

 if ENABLE_AUDIOPLAYER
 libneutrino_gui_a_SOURCES += \
-       audioplayer.cpp audioplayer_setup.cpp
+       audioplayer.cpp audioplayer_setup.cpp mediaplayer_setup.cpp
 endif

 if ENABLE_PICTUREVIEWER
 libneutrino_gui_a_SOURCES += \
-       pictureviewer.cpp pictureviewer_setup.cpp
+       pictureviewer.cpp pictureviewer_setup.cpp mediaplayer_setup.cpp
 endif

 if ENABLE_MOVIEPLAYER
-libneutrino_gui_a_SOURCES += moviebrowser.cpp movieplayer_setup.cpp
+libneutrino_gui_a_SOURCES += moviebrowser.cpp movieplayer_setup.cpp mediaplayer_setup.cpp
 if ENABLE_MOVIEPLAYER2
 libneutrino_gui_a_SOURCES += movieplayer2.cpp
 else

[dbt]

Sieht so aus. Wenns passt, mach wie du denkst

Edit. im Patch waren noch einige locales für Warnmeldungen drin und die dazugehörigen Messages in neutrino.cpp. Die habe ich entfernt und den Patch nochmal hochgeladen. Also nicht wundern.

[rhabarber1848]

Zu den Suchlauf-Einstellungen, warum sind diese
jetzt in einem Untermenü der Kanalsuche?
Mir hat das alles-auf-einen-Blick-Menü besser gefallen.

[GetAway]

Laut seife hat sich die Adresse der FSF geändert. Solltest Du direkt mit reinnehmen.

Foundation, 51 Franklin Street, Fifth Floor Boston, MA 02110-1301, USA.

[rhabarber1848]

der Vorschlag Save in den jeweiligen Untermenüs verfügbar zu machen, natürlich so, dass nur das gespeichert wird, was vom jeweiligen Menü bezweckt wird

Diese Idee finde ich nicht gut.

Meine Vorgehensweise beim Konfigurieren von Neutrino ist so,
dass ich viele Einstellungen quer durch alle Menüs vornehme
und zum Schluß im Einstellungshauptmenü einmal "Einstellungen
speichern" aufrufe und alles wird gespeichert.

Zusätzliche "Einstellungen speichern"-Menüpunkte verwirren
nur den Benutzer, besonders dann, wenn nur Teilmengen
gespeichert werden, die zudem die Flashchips durch
unnötige Schreibzugriffe belasten, da statt eines Schreib-
vorgangs nun mehrfach gespeichert wird. Es kann auch dazu
führen, dass Benutzer nicht in allen Einstellungsuntermenüs
und überhaupt nicht im Einstellungshauptmenü "Einstellungen
speichern" aufrufen, was dazu führt, dass erst beim Herunter-
fahren von Neutrino die Optionen gespeichert werden.
Manche Leute richten zapit allerdings so ein, dass die config-
Dateien hochgeladen werden und dann wird der Stecker gezogen
im Glauben, Neutrino hätte alle Optionen gespeichert....

Nun könnte man einwenden, das Speichern in den Untermenüs
ist optional und die kompletten Einstellungen werden weiterhin
im Einstellungshauptmenü durch "Einstellungen speichern"
abgespeichert, nun, dann kann man doch auf die "Einstellungen
speichern"-Menüpunkte in den Einstellungsuntermenüs verzichten,
oder?

[dbt]

Zu den Suchlauf-Einstellungen, warum sind diese
jetzt in einem Untermenü der Kanalsuche?
Mir hat das alles-auf-einen-Blick-Menü besser gefallen.

Hatte ich schon mal erklärt:
http://forum.tuxbox-cvs.sourceforge.net ... 24#p366524

An der Save-Geschichte ändert sich auch nichts, war auch nur eine Idee.
Edit: allerdings sind saveSetup() und loadsetup() auch so aufgeblähte Monster in neutrino.cpp. Das ist aber eine andere Geschichte

[rhabarber1848]

weniger genutzte Menüs sollte man nicht unnötig einblenden

Um vernünftige Umschaltzeiten zu haben, ist ein doppelter
Scan mit beiden Schnellscan-Einstellungen nötig. Deshalb
muss das neue Untermenü permanent genutzt werden.
Mir würde es schon reichen, wenn die Schnellscan-
Einstellung direkt im Kanalsuche-Menü drin wäre.

[dwilx]

Um vernünftige Umschaltzeiten zu haben, ist ein doppelter
Scan mit beiden Schnellscan-Einstellungen nötig.

Ist das bei Kabel üblich (habe nur Sat)? Ist mir neu ? Kannst du das etwas erläutern? Wenn das unbedingt eingestellt sein muss, dann könnte man doch von vornherein in Abhängigkeit vom Boxtyp die Defaultwerte passend einstellen?

[rhabarber1848]

Diese Posting betraf nur die Kanalsuche und findet sich jetzt hier:
http://forum.tuxbox-cvs.sourceforge.net ... 12#p371512

[dwilx]

Wenn das so effektiv ist, kann man das nicht gleich irgendwie einbauen, dass das beim Scan von selbst gemacht wird, etwa als Option "doppel-Scan" (ich weiß, blöder Name, ist nur sinngemäß)?