Commit-Wünsche für das CVS II
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Und eine weitere kleine Änderung für themes.cpp. Ich finde es passender, wenn vor den Theme-Einträgen das OK-Button-Symbol angezeigt wird.
Link entfernt, da Patch im CVS
Link entfernt, da Patch im CVS
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
committed: http://article.gmane.org/gmane.comp.vid ... x.scm/3294Gaucho316 hat geschrieben:themes_show-ok-button-on-selected-item_2012-03-15_2011.diff
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Ich habe noch ein paar Stellen in Menüs rausgesucht, von denen ich denke, dass das OK-Button-Symbol besser passt. Dort wird nämlich über den MenuForwarder eine Aktion direkt ausgeführt, ohne dass sich etwas öffnet, wie man es bei dem Pfeil-Symbol erwarten würde.
Link entfernt, da Patch im CVS
Link entfernt, da Patch im CVS
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
committed:Gaucho316 hat geschrieben:neutrino_show-ok-button-on-some-menuforwarders_2012-03-18_2022.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3315
http://article.gmane.org/gmane.comp.vid ... x.scm/3316
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Ich habe mal wieder drei Patches zum Einchecken. Alle drei sind getestet und funktionieren.
Mir ist aufgefallen, dass an einigen wenigen Stellen in Menu Widgets am Ende noch ein Menu Separator rangehängt wird. Ich habe das mal rausgesucht und die Menu Separators entfernt.
Link entfernt, da Patch im CVS
Dabei ist mir wiederum ein Speicherleck im Audioplayer aufgefallen. Dort wird mit new ein Objekt vom Typ CMenuSelectorTarget angelegt und nie wieder gelöscht.
Link entfernt, da Patch im CVS
Und dabei habe ich die Klasse CMenuSelectorTarget überhaupt erst entdeckt, die sich wunderbar für das Startpositionsauswahlmenü des Moviebrowsers eignet. Dann kann nämlich der wenig praktikable Code entfallen, der über getSelectedLine das gewählte Element herausfindet und bei dem man immer darauf achten muss, dass man die Intro Items beachtet und nicht mit in die Berechnung einbezieht.
Link entfernt, da Patch im CVS
P.S.: Bitte das hier auch gleich einchecken:
http://forum.tuxbox.org/forum/viewtopic ... 05#p382505
Edit: Ich habe den Patch für das Startpositionsauswahlmenü des Moviebrowsers eben noch um eine Kleinigkeit ergänzt. Früher gab es mal eine Trennlinie zwischen den Standard- und den Benutzerbookmarks, die aber enfernt werden musste, da sie die Berechnung durcheinander gebracht hat. Nun kann die aber wieder rein, da sie den neuen Code nicht beeinflusst.
Mir ist aufgefallen, dass an einigen wenigen Stellen in Menu Widgets am Ende noch ein Menu Separator rangehängt wird. Ich habe das mal rausgesucht und die Menu Separators entfernt.
Link entfernt, da Patch im CVS
Dabei ist mir wiederum ein Speicherleck im Audioplayer aufgefallen. Dort wird mit new ein Objekt vom Typ CMenuSelectorTarget angelegt und nie wieder gelöscht.
Link entfernt, da Patch im CVS
Und dabei habe ich die Klasse CMenuSelectorTarget überhaupt erst entdeckt, die sich wunderbar für das Startpositionsauswahlmenü des Moviebrowsers eignet. Dann kann nämlich der wenig praktikable Code entfallen, der über getSelectedLine das gewählte Element herausfindet und bei dem man immer darauf achten muss, dass man die Intro Items beachtet und nicht mit in die Berechnung einbezieht.
Link entfernt, da Patch im CVS
P.S.: Bitte das hier auch gleich einchecken:
http://forum.tuxbox.org/forum/viewtopic ... 05#p382505
Edit: Ich habe den Patch für das Startpositionsauswahlmenü des Moviebrowsers eben noch um eine Kleinigkeit ergänzt. Früher gab es mal eine Trennlinie zwischen den Standard- und den Benutzerbookmarks, die aber enfernt werden musste, da sie die Berechnung durcheinander gebracht hat. Nun kann die aber wieder rein, da sie den neuen Code nicht beeinflusst.
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
committed:
http://article.gmane.org/gmane.comp.vid ... x.scm/3322
http://article.gmane.org/gmane.comp.vid ... x.scm/3325
http://article.gmane.org/gmane.comp.vid ... x.scm/3321Gaucho316 hat geschrieben:neutrino_remove-menu-separators-from-the-end-of-menu-widgets_2012-03-31_1832.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3322
http://article.gmane.org/gmane.comp.vid ... x.scm/3323Gaucho316 hat geschrieben:audioplayer_fix-memleak_2012-03-31_1926.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3324Gaucho316 hat geschrieben:moviebrowser_rework-start-pos-selection-menu_2012-04-01_1636.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3325
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Wie immer danke fürs Einchecken.
Und noch einer. Das war ein ganz schönes Stück Arbeit. Ausgehend vom Moviebrowser ist mir aufgefallen, dass im Menü "Film-informationen" die Umlaute und das ß nicht richtig dargestellt und bearbeitet werden können. Der folgende Patch löst das Problem und sorgt außerdem dafür, dass man nun auch Umlaute und das ß in der Bouquet-Verwaltung, für Bookmarks im Movieplayer/-browser, in der EPG-Suche, für Erinnerungs-Timer und für die Beschriftung der Benutzermenüs verwenden kann. Der Patch ist getestet und ich konnte bis jetzt keine Nebenwirkungen erkennen.
Link entfernt, da Patch im CVS
Edit: Ich habe den Patch eben aktualisiert. An der Funktionalität ändert sich nichts. Ich habe nur ein paar Bezeichner umbenannt.
Und noch einer. Das war ein ganz schönes Stück Arbeit. Ausgehend vom Moviebrowser ist mir aufgefallen, dass im Menü "Film-informationen" die Umlaute und das ß nicht richtig dargestellt und bearbeitet werden können. Der folgende Patch löst das Problem und sorgt außerdem dafür, dass man nun auch Umlaute und das ß in der Bouquet-Verwaltung, für Bookmarks im Movieplayer/-browser, in der EPG-Suche, für Erinnerungs-Timer und für die Beschriftung der Benutzermenüs verwenden kann. Der Patch ist getestet und ich konnte bis jetzt keine Nebenwirkungen erkennen.
Link entfernt, da Patch im CVS
Edit: Ich habe den Patch eben aktualisiert. An der Funktionalität ändert sich nichts. Ich habe nur ein paar Bezeichner umbenannt.
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Und noch zwei kleine Patches.
Im Bookmarkmanager (Hauptmenü -> Movieplayer -> Bookmarks) wird in der rechten oberen Ecke das Hilfe-Icon angezeigt, obwohl überhaupt keine Hilfe existiert. Ich habe das Icon deshalb auskommentiert.
Link entfernt, da Patch im CVS
In der EPG-Suche finde ich es schön, wenn der Titel des gewählten Events direkt als Suchwort übernommen wird.
Link zum Patch entfernt
Im Bookmarkmanager (Hauptmenü -> Movieplayer -> Bookmarks) wird in der rechten oberen Ecke das Hilfe-Icon angezeigt, obwohl überhaupt keine Hilfe existiert. Ich habe das Icon deshalb auskommentiert.
Link entfernt, da Patch im CVS
In der EPG-Suche finde ich es schön, wenn der Titel des gewählten Events direkt als Suchwort übernommen wird.
Link zum Patch entfernt
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
committed:Gaucho316 hat geschrieben:neutrino_fix-input-of-german-umlauts-and-special-chars_2012-04-07_1248.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3331
http://article.gmane.org/gmane.comp.vid ... x.scm/3332
http://article.gmane.org/gmane.comp.vid ... x.scm/3333
http://article.gmane.org/gmane.comp.vid ... x.scm/3334
http://article.gmane.org/gmane.comp.vid ... x.scm/3335Gaucho316 hat geschrieben:bookmarkmanager_comment-out-help-icon_2012-04-07_1551.diff
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
Habe ich noch nicht in Aktion testen können, kommt aber nochGaucho316 hat geschrieben:In der EPG-Suche finde ich es schön, wenn der Titel des gewählten Events direkt als Suchwort übernommen wird.
eventlist_use-selected-event-title-as-default-search-word_2012-04-07_1627.diff
PS: Ich würde "m_search_keyword.empty()" rauslassen. Wenn von der Eventliste ausgehend
das Suchmenü geöffnet wird, wird der markierte Sendungstitel übertragen. Wenn das Suchfenster
geschlossen, in der Eventliste eine andere Sendung markiert und erneut das Suchmenü geöffnet
wird, wird der bisherige Sendungsname nicht durch die neu markierte Sendung überschrieben.
Imho ist das nicht intuitiv.
Zuletzt geändert von Gaucho316 am Samstag 14. April 2012, 17:16, insgesamt 1-mal geändert.
Grund: Verlinkung entfernt
Grund: Verlinkung entfernt
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Hatte ich auch zuerst so gehabt. Das Problem ist dann aber, wenn man etwas von Hand eingegeben hat und die Suche wieder öffnet, wird einem das gnadenlos überschrieben. Das wollte ich verhindern.rhabarber1848 hat geschrieben:Ich würde "m_search_keyword.empty()" rauslassen.
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
Stimmt. Alternativvorschlag: Ein hotkey im ansonsten leeren Eingabefeld, dass denGaucho316 hat geschrieben:Hatte ich auch zuerst so gehabt. Das Problem ist dann aber, wenn man etwas von Hand eingegeben hat und die Suche wieder öffnet, wird einem das gnadenlos überschrieben. Das wollte ich verhindern.rhabarber1848 hat geschrieben:Ich würde "m_search_keyword.empty()" rauslassen.
ausgewählten Sendungstitel übernimmt und die bisherigen Eingaben überschreibt?
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Halte ich für zu aufwendig. Ich habe mir da mal etwas einfacheres überlegt. Jetzt wird vor dem Aufruf des Suchmenüs das letzte automatisch gesetzte Suchwort mit dem tatsächlichen verglichen. Sind sie gleich, wurde es folglich nicht geändert und kann wieder automatisch gesetzt werden. Wurde es geändert, bleibt es dagegen so, wie es ist.
Link zum Patch entfernt
Link zum Patch entfernt
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Einen Fehler habe ich in der Klasse CStringInput noch gefunden. Wenn man einen String (vom Typ std::string) bearbeitet, der länger als das Eingabefeld ist, und das Bearbeiten mit der Home-Taste abbricht, wird der String trotzdem auf die Länge des Eingabefeldes gestutzt. Dieses Fehlverhalten kann man sehr gut bei der Bearbeitung der Film-Informationen im Moviebrowser beobachten.
Link entfernt, da Patch im CVS
Außerdem sollte der alte Wert des bearbeiteten Strings (vom Typ char*) wiederhergestellt werden, wenn das Bearbeiten "von außen" abgebrochen wird.
Link entfernt, da Patch im CVS
Link entfernt, da Patch im CVS
Außerdem sollte der alte Wert des bearbeiteten Strings (vom Typ char*) wiederhergestellt werden, wenn das Bearbeiten "von außen" abgebrochen wird.
Link entfernt, da Patch im CVS
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Ich habe nun doch wieder den Test auf m_search_keyword.empty() aus der ersten Version eingebaut. Dann wird zusätzlich das Suchwort vorausgefüllt, wenn es leer ist. Das sollte es nun aber endgültig sein.Gaucho316 hat geschrieben:Jetzt wird vor dem Aufruf des Suchmenüs das letzte automatisch gesetzte Suchwort mit dem tatsächlichen verglichen. Sind sie gleich, wurde es folglich nicht geändert und kann wieder automatisch gesetzt werden. Wurde es geändert, bleibt es dagegen so, wie es ist.
Link entfernt, da Patch im CVS
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Ich denke, dass man das auch für die Klasse CExtendedInput machen sollte.Gaucho316 hat geschrieben:Außerdem sollte der alte Wert des bearbeiteten Strings (vom Typ char*) wiederhergestellt werden, wenn das Bearbeiten "von außen" abgebrochen wird.
Link entfernt, da Patch im CVS
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Einen hab ich noch. In den Klassen CMenuOptionNumberChooser, CMenuOptionChooser und CMenuOptionStringChooser ist die Berechnung der Startposition des Optionswerts falsch. In der Klasse CMenuForwarder wird das richtig gemacht. Deshalb habe ich das von da übernommen. Aufgefallen ist mir das bei der Bearbeitung des Genres in den Film-Informationen des Moviebrowsers. Geht man dort alle Optionen durch, bleibt irgendwann links ein Zeichenrest übrig.
Link entfernt, da Patch im CVS
Link entfernt, da Patch im CVS
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
15 Downloads und keine negative Meldung. Ich würde sagen, der Patch für die LCD-Anzeige kann auch ins CVS.
http://forum.tuxbox.org/forum/viewtopic ... 93#p382793
http://forum.tuxbox.org/forum/viewtopic ... 93#p382793
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
Einer davon war ich Ich konnte den Patch und die anderen aber noch nichtGaucho316 hat geschrieben:15 Downloads und keine negative Meldung.
testen, die commits mache ich nach erfolgreichen Test am Wochenende
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
committed:
http://article.gmane.org/gmane.comp.vid ... x.scm/3366Gaucho316 hat geschrieben:stringinput_only-change-value-string-if-ok-pressed_2012-04-11_0822.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3367Gaucho316 hat geschrieben:stringinput_restore-old-value-if-cancelled_2012-04-11_1507.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3368Gaucho316 hat geschrieben:stringinput_ext_restore-old-value-if-cancelled_2012-04-13_1420.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3369Gaucho316 hat geschrieben:eventlist_use-selected-event-title-as-default-search-word_2012-04-12_2308.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3370Gaucho316 hat geschrieben:menue_fix-start-pos-calculation-of-option-value_2012-04-14_1900.diff
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Durch einen Thread im DBox2World-Forum bin ich darauf aufmerksam geworden, dass in der EPG-Liste bei Sortierung nach dem Titel Einträge mit demselben Titel nach der Event-ID sortiert werden. Das ist natürlich völlig sinnfrei. Ich habe das mal schnell so umgebaut, dass dann nach der Startzeit sortiert wird.
Link entfernt, da Patch im CVS
Link entfernt, da Patch im CVS
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
committed to CVS: http://article.gmane.org/gmane.comp.vid ... x.scm/3374Gaucho316 hat geschrieben:eventlist_sort-events-with-same-title-by-starttime_2012-04-26_0857.diff
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Da ich auch immer ein Auge auf die Commits im Coolstream GIT werfe, habe ich mal zwei Patches übernommen, die auch für Neutrino wichtig sind, weil sie Speicherlecks schließen.
CLocaleManager: delete char array in destructor properly
Link entfernt, da Patch im CVS
CPersonalizeGui: fix memleak in addSeparator()
Link entfernt, da Patch im CVS
CLocaleManager: delete char array in destructor properly
Link entfernt, da Patch im CVS
CPersonalizeGui: fix memleak in addSeparator()
Link entfernt, da Patch im CVS
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Der zweite Patch von gestern war nur ein Teil eines Patches aus dem Coolstream GIT. Hier nun der Rest, der dafür sorgt, dass auch die Personalisierungseinstellungen für das Haupt-, das Einstellungs- und das Servicemenü mit einem Druck auf die d-box-Taste komplett (d.h. inklusive aller vorher geöffneten Menüs) verlassen werden.
CPersonalizeGui: close all menus on dbox key
Link entfernt, da Patch im CVS
Und dann habe ich noch einen Patch für die Personalisierung, der diesmal aber von mir ist. Wenn man im Personalisierungsmenü etwas geändert hat, wird man ja beim Verlassen des Menüs gefragt, ob man die Änderungen übernehmen möchte. Wählt man "Nein", wird man dann gefragt, ob man die Änderungen verwerfen möchte. Wählt man hier "Ja", werden die Änderungen allerdings nicht, wie erwartet, zurückgesetzt. Außerdem landet man dann wieder im Personalisierungsmenü, was für mich keinen Sinn ergibt. Man hat ja schließlich vorher die Home-, Links- oder d-box-Taste gedrückt bzw. "Zurück" im Menü gewählt.
CPersonalizeGui: fix settings restore
Link entfernt, da Patch im CVS
CPersonalizeGui: close all menus on dbox key
Link entfernt, da Patch im CVS
Und dann habe ich noch einen Patch für die Personalisierung, der diesmal aber von mir ist. Wenn man im Personalisierungsmenü etwas geändert hat, wird man ja beim Verlassen des Menüs gefragt, ob man die Änderungen übernehmen möchte. Wählt man "Nein", wird man dann gefragt, ob man die Änderungen verwerfen möchte. Wählt man hier "Ja", werden die Änderungen allerdings nicht, wie erwartet, zurückgesetzt. Außerdem landet man dann wieder im Personalisierungsmenü, was für mich keinen Sinn ergibt. Man hat ja schließlich vorher die Home-, Links- oder d-box-Taste gedrückt bzw. "Zurück" im Menü gewählt.
CPersonalizeGui: fix settings restore
Link entfernt, da Patch im CVS
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Und noch ein Patch aus dem Coolstream GIT, der ein Speicherleck in der Klasse CKeybindSetup schließt. Das ist mir wohl damals durch die Lappen gegangen, als ich alle Menüklassen durchgegangen bin.
CKeybindSetup: fix memleak
Link entfernt, da Patch im CVS
CKeybindSetup: fix memleak
Link entfernt, da Patch im CVS