Conversation
|
Thanks @joeyklee 🤘 Let me know if I need to make any changes. |
| <script src="https://unpkg.com/ml5@0.4.0/dist/ml5.min.js" type="text/javascript"></script> | ||
|
|
||
| <style></style> | ||
| <script src="sketch.js"></script> |
There was a problem hiding this comment.
Correct me if I'm mistaken, but we should always try to leave the sketch.js at the end of the body tag to make sure that the DOM elements are loaded before calling the script.
This example will work fine this way, but for consistency, we should just always call script.js at the end of the index.html.
There was a problem hiding this comment.
Thanks for the reply!
That sounds about right, I'll have time to hack on it later today 🤘
There was a problem hiding this comment.
Thanks a bunch! Let me know whenever you're ready and I can review. If you're able to resolve any of the merge conflicts that would be great, otherwise I can also take care :)
Many thanks!
|
Hey @justsml! We're archiving this repository and moving its contents into the core ml5-library repository (more context on issue #809 and pull request #831). We appreciate your contributions and would love it if you'd consider reopening this pull request on the ml5-library repo! Thanks! |
Dear ml5 community, 👋
I'm making a Pull Request(PR). Please see the details below.
→ Step 1: Which branch are you submitting to? 🌲
→ Step 2: Describe your Pull Request 📝