maadc / studienarbeit_gruppe1

Studienarbeit im Fach Web Engineering
http://sta.schranz.io
0 stars 4 forks source link

So in etwa müsste das aussehen #38

Closed fonzerelly closed 4 years ago

fonzerelly commented 4 years ago

Hallo Markus,

also hier mal wie ich mir das in etwa vorstelle. Allerdings sind die pads verschwunden. Das war aber in Deiner Version auch schon der Fall.

Aber so kann man main.js in zwei Dateien calc.js und dom.js aufteilen und beide separat testen.

Dieser Code am ende von calc: if (window) {...}

if (module) {...}

ist Code den man schreiben muss, wenn man will, dass die eigenen Funktionen im Browser als auch im Node laufen.

Das könnte man ich evtl sparen, wenn man sich mit einem Build-Tool wie WebPack auseinander setzen würde.

Kuck bitte auch nochmal über die checkKeys-Funktionen, die könnte man soviel verständlicher und einfacher noch schreiben. Warum handelt ihr die key-Downevents nicht direkt und spart Euch diese Weiterleitung über checkPassedKeys?

Übrigens das document, dass eigentlich hätte Fehlen müssen, kommt wohl aus Jest selbst, um JavaScript-gebaute Komponenten dort direkt abzulegen.

Bis dann Christian

maadc commented 4 years ago

@fonzerelly Wieso sollen wir mitten im Sprint unsere Dateien aufteilen? Ich habe bereits ein Issue #34 erstellt, dass genau dieses "Problem" beinhaltet. Können wir das nicht im nächsten Sprint machen?

Brechen wir nicht die Scrum-Regeln, wenn wir im Sprint Aufgaben übernehmen, die wir vorher nicht besprochen haben?