Needed to comment out the line with p5.sound to make sketch work ok
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 24k
- Forks
- 3.8k
- Avg merge
- 3d 16h
- Merged PRs (30d)
- 25
Description
p5.js version
1.9.4
Web browser and version
142.0.7444.60 (Official Build) (x86_64)
Operating system
MacOSX/Android
Steps to reproduce this
Hi, just wanted to document this in case it's useful. The sketch worked fine for several browser and webserver combinations. But for chrome and one of the webservers it didn't, which was confusing for a beginner like me.
Did fix it, by commenting out the second line here. With second line commented out it works for all browser and webserver combinations. BTW this code came from an example, it's basically my first p5js sketch and this code is from the example I started with:)
<script src="https://cdnjs.cloudflare.com/ajax/libs/p5.js/1.9.4/p5.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/p5.js/1.9.4/addons/p5.sound.min.js"></script>
Cheers,
Noah
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reproducing the report in Chrome 142 with the two CDN script tags shown, especially the p5.sound.min.js addon, on the affected browser and web-server combinations. Done means identifying a reliable cause and documenting or confirming the supported loading setup; the issue does not name a repository file or test.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend, web-dev
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100