p5.js
p5.js copied to clipboard
Replace console.error with friendly error system
Increasing Access
Many functions inside src/webgl and lib/addons currently uses console.error to report errors. It would be more user-friendly and consistent with the p5.js environment to replace these calls with the friendly error system (FES).
Most appropriate sub-area of p5.js?
- [X] Accessibility
- [ ] Color
- [ ] Core/Environment/Rendering
- [ ] Data
- [ ] DOM
- [ ] Events
- [ ] Image
- [ ] IO
- [ ] Math
- [ ] Typography
- [ ] Utilities
- [X] WebGL
- [ ] Build Process
- [ ] Unit Testing
- [ ] Internalization
- [X] Friendly Errors
- [ ] Other (specify if possible)
Feature enhancement details
For example, take parseASCIISTL function in webgl/loading.js,
I have triggered an error condition and the results,
with console.error:
with FES:
It would be more user-friendly and consistent with the p5.js environment to replace these calls with the friendly error system (FES).
Welcome! 👋 Thanks for opening your first issue here! And to ensure the community is able to respond to your issue, please make sure to fill out the inputs in the issue forms. Thank you!
related PRs and Issues: #6098 and #5649 .
@davepagurek if you can review this. I am willing to work on this issue. Please assign.
can I get to work on this issue? this will be my first contribution in processing community
where to find this issue I forked the code base in local system??
Do we also have to change the throw new error keywords or only the console.error
@shashank-iitbhu please accept the pull request
@davepagurek Hey I would like to work on this issue. Can you assign me?