Gewinn- bzw. Verlustdarstellung

[GoogleCodeExporter]

Es wäre schöner, wenn im Sieg oder Niederlage Fall anstelle eines Dialoges 
(der wohlbermerkt in allen Browsern unterschiedlich schlecht aussieht) ein von 
uns einheitlich definierbares div erscheint.

Dieses div soll über eine CSS Klasse definierbar sein und einen großen Teil 
des canvas überdecken. (horizontal und vertikal auf dem canvas zentriert) 

In dem div können später dann auch Statistiken mitangezeigt werden.

Über design des div muss gemeinsam entschieden werden, jedoch kann initial ein 
intuitives design gewählt werden.

Original issue reported on code.google.com by ichShane on 11 Nov 2011 at 8:04

[GoogleCodeExporter]

Original comment by ichShane on 25 Nov 2011 at 9:12

• Changed state: Started

[GoogleCodeExporter]

Find das gut, aber ein paar Dinge sind mir aufgefallen:

- Wenn ich mehrere Spiele hintereinander Spiele, dann habe ich mehrere gewinn 
bzw. verlustmeldungen in dem Fenster. Das liegt daran, dass du immer nur mit 
append Text hinzufügst, allerdings muss dieser vor der nächsten darstellung 
wieder gelöscht werden.

- Du hast die id des new Game button in eine class umgewandelt. Dies ist 
eigentlich nicht ok. Da du vergessen hast die dDefinitionen für die id newGame 
in der css Klasse auch in eine class Definition umzuwandeln, hat sich das 
Verhalten des Button geändert. Änderungen von id zu class bitte nur nach 
rücksprache, da sich unter Umständen, wie in diesem Fall, Codeänderungen 
daraus ergeben. Außerdem hast du dem Button eine neue Klasse zugewiesen 
(newGameTop) zu der ich keine Definition finde und mir auch nicht bewusst ist, 
wofür die gut sein soll.

- Du hast an manchen Stellen Dokumentation und zugehörigen Code voneinander 
getrannt, indem du Code dazwischen gepackt hast, das sollte auf keinen Fall 
geschehen, da die Dokumentation dann natürlich unbrauchbar wird.

Ich werde die von mir festgestellten Probleme eben selbst beheben.

Der nächste Schritt wäre das Darstellen der Statistiken :)

Hierfür wäre es evtl sinnvoll den Statistiken Berech zu vergrößern, da es 
sich insgesamt um ein paar mehr Werte handelt.

Original comment by ichShane on 29 Nov 2011 at 7:56

[GoogleCodeExporter]

Der erste Punkt ergab sich daraus, dass der Text fehlerhaft entfernt wurde. 
Wenn man alle p Elemente mit der class wlmessage entfernen möchte, so lautet 
der Befehl: $('p').remove(".wlmessage");

Den zweiten Punkt habe ich rückgängig gemacht. Der Button New Game hat jetzt 
wieder die id newGame und keine css klasse. dies ist gewünscht, da der Button 
ein individuelles Verhalten haben soll.

Den dritten Punkt habe ich ebenfalls korrigiert.

Zusätzlich habe ich das Kontextmenü des Messagefeldes unterdrückt.

Original comment by ichShane on 29 Nov 2011 at 8:04

[GoogleCodeExporter]

Ich habe die Statistik soweit fertiggestellt. 

-Die Combobox im Statistik Feld stimmt nun mit der Aktuellen Schwierigkeit 
überein.

-Nach einem Spiel werden die aufgedeckten Felder in Prozent angegeben sowie die 
Benötigte Zeit und der Schwierigkeitsgrad.

-In dem Statistik Feld werden nun die Spiele, Zeit und Prozent jedes 
Schwierigkeitsgrades als gesamt Summe angezeigt.

-Bei der Auswahl einer anderen Schwierigkeit ändern sich die Statistiken  
gleich mit. 

Original comment by chris-bl...@freenet.de on 30 Nov 2011 at 12:04

• Changed state: ForReview

[GoogleCodeExporter]

Original comment by ichShane on 1 Dec 2011 at 7:40

• Changed state: Review

[GoogleCodeExporter]

Ich habe Schreibfehler in Ausgaben korrigiert. Außerdem habe ich die 
Generierung der Ausgabewerte angepasst, sodaß diese den gleichen Stil 
verfolgen und entsprechende Methoden zur Formatierung verwenden.

Außerdem ist mir eingefallen, dass wir noch die Bestzeiten unterbringen 
müssen.

Original comment by ichShane on 1 Dec 2011 at 8:02

• Changed state: Started

[GoogleCodeExporter]

Aufgegeben muss in abgebrochen umbenannt werden

Original comment by ichShane on 4 Dec 2011 at 7:43

[GoogleCodeExporter]

Gewinntext ist unzureichend

Original comment by ichShane on 4 Dec 2011 at 7:46

[GoogleCodeExporter]

Ich habe die Texte angepasst, sodass diese einheitlich aussehen.
Aufgegeben heißt jetzt abgebrochen
Die bestzeit wird unterhalb der gewinnstatistiken angezeigt.

Original comment by ichShane on 7 Dec 2011 at 1:31

• Changed state: Fixed