dynamischer message dialog mit javascript und jquery

[mats]

Hi,

ich versuche einen message dialog dynamisch zu erzeugen welcher einen input zur text eingabe zu verwendet.

warum wird changeLevelClicked nicht aufgerufen wenn ich auf ok klicke?
anstatt bekomme ich:

Uncaught ReferenceError: changeLevelClicked is not defined

hier ist das beispiel:
http://jsfiddle.net/DYbwb/1173/

[mats]

so gehts nun: [url]http://jsfiddle.net/DYbwb/1174/ [/url]

bitte um feedback... wie kann man das noch anders/besser machen abgesehen von der optimierung des codes?

[Shade Of Mine]

Besser machen geht mit Templates und dann gleich ein MVC System. zB Backbone oder React oder was auch immer. Gibt ja Frameworks dafür wie Sand am Meer.

Ansonsten: templates statt händischem Markup und natürlich events statt callback

[mats]

Hi Shade Of Mine, kannst du zeigen wie du es mit einem template bauen wuerdest? ich moechte vorerst kein Backbone oder React verwenden.

[mats]

@Shade Of Mine:
ich habe nun die css des dialoges in ein eigenes file gegeben: http://jsfiddle.net/mx2sfpwf/5/

wie kann ich den rest in ein template verpacken? kannst du es kurz anhand eines jsfiddle zeigen?

[Shade Of Mine]

Wie gesagt: ich würde statt dem callback changeLevelClicked ein Event verwenden.
Und statt den HTML Code in Javascript so zu schreiben, einfach per Underscore Template http://underscorejs.org/#template oder dergleichen.

Ich würde dir aber raten mit einem guten Framework anzufangen, dann lernst du nämlich wie es richtig geht. Dein Code ist zwar korrekt aber halt auch kilometer weit von gutem Code entfernt.

Ein Dialog ist eine View. Also schnapp dir dein lieblings MVC Framework und implementiere den Dialog als View.