Feature/merkliste

[KWich]

Neues Feature Merkliste: Stellt eine Merkliste bereit, dies es erlaubt Filme zu markieren und mit Kommentaren zu versehen zum späteren anschauen oder herunterladen:

Filme können in der Filmliste für die Merkliste entweder über Buttons oder Kontextmenü selektiert werden., alle gemerkten Filme können entweder per Filter oder über ein eigenes Fenster dargestellt werden.

Im Merklistenfenster können zusätzliche Anmerkungen und ein Verfügbarkeitsdatum eingegeben werden. Der entsprechende Dialog beinhaltet eine Funktion die versucht das Verfügbarkeitsdatum von der Filmwebseite zu holen (geht nicht bei allen Sendern), ansonsten kann das Datum im Dialog eingegeben werden. Falls der Film nur noch wenige Tage verfügbar ist. wird dieses in der Merkliste hervorgehoben.

Gemerkte Filme werden in eigener Liste gehalten (und als JSON Datei gespeichert), die Einträge sind dynamisch mit der Filmliste verknüpft. Filme die aus der Filmliste verschwinden, werden in der Merkliste markiert, bleiben aber weiter erhalten bis sie gelöscht werden.

Hinweis: Eine derartige Funktion wurde unter Issue#230 diskutiert.

Bei Fragen bitte direkt kontaktieren, falls mehr Dokumentation gewünscht kann ich diese erstellen.

[derreisende77]

Habe mir den PR nur kurz angesehen und muß sagen: Klasse arbeit, ist ein cooles Feature was aufgenommen werden sollte!

Allerdings habe ich ein paar Anmerkungen und mindestens einen echten Bug gefunden:

• [ ] Der "Schließen" Button in der Statuszeile ist mit keinem User Interface Style Guide kompatibel. Ich würde ihn ersatzlos streichen da die Fenster auf allen OS einen Schließen Mechanismus bieten und dieser hier redundant ist.
• [ ] Der Toolbar Button "Merkliste anzeigen" sollte im Menü "Ansicht" auch hinterlegt sein, wir haben User die ohne den Menüeintrag Features nicht finden.
• [ ] BUG? Die Farbmarkierung in Blau der Filme erschließt sich mir nicht 100%. Im nachfolgenden Screenshot sind alle Filme gebookmarked aber der vorletzte ist nicht gehighlighted. Warum? (kann es hier die Kombination mit dem blau markiertem "neuen" Film sein?)
• [ ] Gebookmarkte Filme besitzen das Icon blau gefüllt. Wie auf dem Screenshot oben auch erkennbar markiert macOS Zeilen in Blau (letzter Eintrag). Blau auf Dunkelblau ist eine bescheidene Kombination und das Icon sollte unter macOS statt blau weiß gefüllt werden wenn es highlighted ist.
• [ ] Beim bookmarken selektierter Filme geht die Auswahl der Filme nach der Aktion verloren. Die meisten anderen Befehle (z.B. als gesehen/ungesehen markieren) restoren die gewählten Zeilen danach wieder. Das Verhalten der neuen Action sollte equivalent umgesetzt werden.
• [ ] BUG: Einige Filme bookmarken. Im Anschluß Filter „gemerkte Filme anzeigen“ setzen und Programm beenden. Beim erneuten Start bleibt der Filter gesetzt, aber die dargestellte Anzahl an Filmen variiert bei jedem Start und ist meist falsch. Ausgangspunkt sind die 5 gebookmarkten Filme aus dem Screenshot oben. Der erste Neustart mit gesetztem Filter führt zu folgendem Bild: Der Neustart darauf zu folgendem Bild:

Wenn der/die Bugs behoben sind werde ich den PR nach develop mergen damit er in die nächste Version einfließen kann.

[derreisende77]

PS: Was meinst Du mit „direkt kontaktieren“?

[KWich]

PS: Was meinst Du mit „direkt kontaktieren“? Bei schnellen Rückfragen einfach meine e-Mail aus den GIT Commits benutzen.

[KWich]

Ich habe den Code entsprechend des Reviews angepasst:, sollte jetzt passen

• Schließen Button entfernt (war ein Überbleibsel der ersten Dialogs)

• Menü "Ansicht" enthält jetzt auch einen Menü Eintrag für die Merkliste

• Farbmarkierung: keine Änderung: Wie schon vermutet werden all gemerkten Filme farbig hinterlegt, wenn keine andere Farbmarkierung gesetzt wird (wie neu, gesehen, usw.) Gemerkt hat die niedrigste Priorität, der Zustand kann auch am Icon abgelesen werden. (Die Farbe kann übrigens in den Einstellungen gewählt werden)

• Blaues Icon für gemerkte Filme, ist jetzt gelb, wenn Eintrag selektiert ist.

• Auswahl bleibt nach Aktion erhalten, habe das auch in der Merkliste angepasst wo sinnvoll.

• BUG Filter beim Start: Grund ist eine Race Kondition, welche vor allem beim automatischen Laden und Abarbeiten der Blacklist auftreten kann: Die Merkliste ist noch nicht mit der Filmliste synchronisiert, da noch auf das "ListenerFilmeLadenEvent" gewartet wird, welches die Synchronisierung auslöst, aber die Filmliste wird schon angezeigt. Das es im Prinzip nicht viel Sinn macht diesen Filter beim Programmende zu speichern, sondern eher für Verwirrung sorgt, habe ich die Speicherung dieser Filter in der Konfiguration entfernt, um das Problem zu lösen.

Zusätzlich ist mir aufgefallen das der Button und die entsprechende Checkbox im Filter zum selektieren der Merkliste asynchron waren, habe ich noch erweitert.