processing / processing/p5.js-web-editor
Attempting to stringify large variable results in an error log
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 1.7k
- Forks
- 1.7k
- Avg merge
- 3d 4h
- Merged PRs (30d)
- 8
Description
p5.js version
dev-2.0 branch
What is your operating system?
None
Web browser and version
Chrome
Actual Behavior
Because of an attempt to stringify a large variable, error is logged in browser.
Expected Behavior
Do not try to stringify objects that are [expected to be] excessively large
Steps to reproduce
Observed in: https://github.com/processing/p5.js/pull/7693
Notes from the above thread:
"that's a p5 web editor bug, it's trying to stringify the whole font including parsed font data, which is too large of an array."
"that would be great thanks! we may want to just add a special case there for p5.Font to ignore some propertiesI feel like we had similar issues in the past with vectors having a reference in them to the p5 instance"
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.
Assessment
This issue has not been assessed yet.