fr3ddy / webengeneering Goto Github PK

Es muss irgendwo abgespeichert sein, wie ein Film bewertet wurde. Sonst muss dies beim filtern jedesmal neu errechnet werden.

Vor dem Hintergrund, dass wir die Liste filtern und sortieren können sollen, fände ich es praktisch, wenn die Kopfzeile der Tabelle nicht verschwindet, sobald man hoch und runter scrollt.
Daher die Frage, ob wir die Liste selbst scrollbar machen wollen und nicht die gesamte Seite?

Die Liste kann nach Filmname und Gesehen / nicht gesehen sortiert werden und nach Bewertung sortiert werden

Focus ist dauerhaft auf Filmtitelfeld und verschwindet auch nicht, wenn irgendwo anders hingeklickt wird

Gehe die HTML File(s) durch und optimiere sie

Unterscheide zwischen angemeldeten und nicht angemeldeten Benutzern. Die angemeldeten sollen Filme hinzufügen, bearbeiten, löschen und sehen können. Die nicht angemeldeten Benutzer sollen die Filme nur sehen können.

JavaScript/jQuery durchgehen und kontrollieren, optimieren, aufräumen, kommentieren...

$('#submitLoginButton').on('click', function(event) {

überarbeiten

Bisher werden die Ids für die TableRows (tr-X) anhand der letzten Zeile in der Tabelle zugewiesen. Wenn die letzte Zeile also die ID tr-5 hat, so bekommt die neu Zeile die ID tr-6.
Problem: Ist die Tabelle sortiert ist die letzte Zeile nicht unbedingt diejenige mit der höchsten ID. IDs kommen nun also doppelt vor!!! Was sollen wir tuen?

Hinzufügen in Zeile 470 & 474

Gesehen / Nicht gesehen nur im Eingeloggten modus sichtbar oder?
sollte ja nur für die spezifische person sein!

Es soll möglich sein einen Film aus der Liste löschen zu können. Dabei muss der Benutzer den Löschvorgang bestätigen

Gesehene Filme sollen mit 1 bis 5 Sternen bewertet werden können.

• Die Sicht für das Hinzufügen von Filmen soll immer mit leeren Sternen beginnen.
• In der Liste sollen die Sterne nur angezeigt, aber nicht bearbeitet werden können.
• In der bearbeiten Sicht, soll die aktuelle Bewertung aus der Liste beim Öffnen in die Sicht übernommen werden und beim Ändern in die Liste gespeichert werden

Nur bewerten durch Klick ansonsten leer. Titel hinzufügen (schlecht, in Ordnung, gut, OMG)

CSS File durchgehen und überprüfen, dass keine unnützen Zeilen vorhanden sind

das es halt gut aussieht!

JavaScript/jQuery durchgehen und kontrollieren, optimieren, aufräumen, kommentieren...

Eine Fehlermeldung soll erscheinen, dass der Film nicht in der OMDB (oder was auch immer wir nutzen) gefunden wurde

Weißt ja

Die Liste kann nach gesehen/nicht gesehen gefiltert werden.

1. bessere Benennungen für Buttons und Methoden rund ums filtern und sortieren
2. Filtern nach Titel und Bewertung

Kann es vorkommen, dass mehrere Filme zu dem Titel gefunden werden, den wir mitgeben? Falls ja, dann sollte eine Auswahl erscheinen, damit der Benutzer den Film auswählen kann, den er tatsächlich gemeint hat

Mit Hilfe von dem Parameter s, sucht man bei OMDBAPI Filme. Da findet man, dann auch mehrere Filme z.B. der Unterschied zwischen
(1) http://omdbapi.com/?t=Pirates+of+the+caribbean&s=Pirates+of+the+caribbean
und
(2) http://omdbapi.com/?t=Pirates+of+the+caribbean

In Fall (1) könnten wir eine Liste öffnen, die alle Titel anzeigt, die in dem JSON vorhanden sind. Mit dem ausgewählten Titel wird dann Fall (2) aufgerufen. Das sollte doch eindeutig genug sein, oder?

• Wir sollten auch drauf achten, dass der "Type" im JSON "movie" ist und nicht, wie in Fall (2) "game"!

Wenn man doppelklickt kommt doofes blaues zeug

Was ist mit Filmen wie Der Herr der Ringe, oder Harry Potter und der Halbblutprinz??

CSS File durchgehen und überprüfen, dass keine unnützen Zeilen vorhanden sind

Gehe die HTML File(s) durch und optimiere sie

Wird ein neuer Film hinzugefügt muss kontrolliert werden, dass der Titel nicht leer oder nur mit Leerzeichen gefüllt ist. Außerdem darf ein Film nicht hinzugefügt werden, wenn der Titel des Films schon in der Liste vorhanden ist (=no duplicates). Das muss dem Benutzer kenntlich gemacht werden

Zeige hinzugefügte Filme in einer Liste/Tabelle an

Ist man angemeldet und drückt oben rechts auf den Logout-Knopf erscheint das Login-Fenster. Sollte da nicht lieber einfach der Logoin-Knopf wieder ersccheinen und bei Klick auf den erst das Login-Fenster?

Was haltet ihr davon, wenn der Name des angemeldeten Benutzers zu sehen ist, anstatt der Logout-Knopf und bei Klick auf den Namen dann der Logout-Knopf als Dropdown erscheint??

Also wir müssen uns was überlegen, was die Templates betrifft. Das customizen von Templates wie jetzt den Filter in der custom.js ist der Horror.

Eingabe, um neuen Film hinzuzufuegen erstellen. Diese soll beinhalten:

• Filmtitel
• Gesehen/nicht gesehen
• Bewertung

Nicht angemeldete Benutzer sehen lediglich den Inhalt der Tabelle mit durchscnittlicher Bewertung und der Info, wie viele User den Film schon gesehen haben. Damit die Bewertung auch aussagekräftiger ist. Sie können aber keine anderen Daten ändern, löschen bzw. erzeugen

Wir sollten die JavaScript Datei möglichst clever in mehre Dateien aufteilen, um den Code insgesamt übersichtlicher zu machen. Da wir nun schon bei über 600 Zeilen Code sind und noch einige dicke Brocken vor uns liegen.

Ist bisher im Laufe der zweiten Code Review angedacht. Falls es nötig wird auch schon früher.

Was meint ihr? Gut? Nicht so gut? Auf keinen Fall?

In der Detailansicht der Filme ist ja, wie abgemacht, oben die Navigationsleiste zu sehen. Allerdings passiert nichts, wenn die Navigationsleiste genutzt wird. Wir müssen also dafür sorgen, dass die Detailansicht über das 'X' und über klicken auf 'Filmverwaltung' oder 'Liste' verlassen werden kann.

Vertikal gibt es einen langen Scrollbalken, der mit der DetailView und dem Home-Container zusammenhängt und sichtbar ist, wenn die Ansichten nicht gerade wechseln

Jede Zeile Code durchgehen

Wurden alle vereinbarent und definierten Ziele erreicht?
Ist die Benutzbarkeit des Programms gewährleistet?
Haben wir unser nötigstes getan, um das Projekt zum Erfolg zu führen?
Sind wir mit dem Endergebnis zufrieden?

Wenn ich einen Film erstelle und das erstemal bewerte wird immer nur der erste Stern gesetzt. Also ich markiere im Modal 5 Sterne und es wird in der Tabelle nachher nur der erste gesetzt. Beim zweitenmal bewerten klappt es dann einwandfrei.

Gehe die HTML File(s) durch und optimiere sie

Werden mehrere Filme mit (fast) dem gleichen Titel gefunden, erscheint eine Auswahlbox, wo der User den Film wählen kann, den er tatsächlich meint.

Sollte dann nicht auch dieser Titel so in die Liste übernommen werden, damit er bei nächsten mal nicht wieder aus der Box auswählen muss?

(1)

• Ist damit zu rechnen, dass es mehr als ein Benutzerkonto gibt?
  Falls ja:
• Sollen die Filme dann Benutzerzugehörig in Datenbank abgespeichert werden?
• Sollen die Filme in der Liste nach Benutzer gefiltert werden können?
• Sollen die Filme, die ein Benutzer hinzugefügt hat, von anderen bearbeitet werden können?

(2) Wann erhalten wir Datenbankzugriff auf die Filme, die in der Liste schon beim ersten Laden der Seite angezeigt werden sollen

!!! Falls es weitere Fragen gibt, bitte hinzufügen !!!

Ruft ein Benutzer die Seite (zum ersten Mal) auf, dann soll an einer Tabellenzeile der Hinweis erscheinen, dass wenn er hier doppelklickt zu einer detailierten Ansicht weitergeleitet wird. Ansonsten weiß der Benutzer nicht von selbst, wie er auf die Detailansicht gelangt.

CSS File durchgehen und überprüfen, dass keine unnützen Zeilen vorhanden sind

Jede Zeile Code durchgehen

JavaScript/jQuery durchgehen und kontrollieren, optimieren, aufräumen, kommentieren...

nur noch das verbinden mit der datenbank

Am Anfang vom Filter steht der Slider in der Mitte und beide Labels sind inaktiv geschalten. Drückt der User auf den Slider fährt er aus der Mitte zu GESEHEN. Klickt der User nochmal fährt er wieder in die Mitte, bei erneutem Klick zu NICHT GESEHEN und erneut bei Klick wieder in die Mitte. Drückt der User auf ein inaktives Label wird der Slider dahin gefahren und das Label aktiviert. Funtkioniert in beide Richtungen und bleibt dabei nicht in der Mitte stehen

Doppelklickt man auf einen Film und direkt danach auf "Liste" im Navigationsmenü wird weder die Home- noch die Detail-Ansicht angezeigt. Dies liegt wahrscheinlich an dem Animations-Event, das gestartet wird, bevor das andere fertig ist.

Es wird vor dem Filmtitel eine Lupe eingefügt, die den User zu Detailansicht leitet. Diese ist vom Stil her wie z.B. der Close-Button, leicht grau und nicht penetrat auffallend

Doppelklick wird dann nicht mehr benötigt.

Film soll in Liste/Tabelle ausgewählt werden können und bearbeitet werden. Anschließend sind die Änderungen in die Liste/Tabelle zu übernehmen