search

Q42 / CT-3000

A tool for teaching Computational Thinking to kids
https://ct-3000.firebaseapp.com/
MIT License
10 stars 3 forks source link

Feature/sound fix ios #20

Closed joeldn closed 8 years ago

guidobouman commented 8 years ago

Zat nog een klein restje van de test code in. Verder vind ik het niet zo mooi dat het via de bovenliggende modules gaat, maar ik snap waarom je het zo doet.

Persoonlijk had ik het geprobeerd met een document.addEentListener in de musicStream.js. Dan creeer je geen tight-coupling. (En geoorloofd omdat het geisoleerd in musicStream.js plaats zou vinden voor een device specific hack.)

Maar for now: Thanks for fixing!

Verwijder die ene regel, dan LGTM ⭐

joeldn commented 8 years ago

En je hebt helemaal gelijk, dus heb het even veranderd @guidobouman !

guidobouman commented 8 years ago

Top, ziet er perfect. uit. LGTM. ⭐