Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Validate input for brush name #10

Closed
Ethspresso opened this issue Nov 24, 2023 · 1 comment
Closed

Validate input for brush name #10

Ethspresso opened this issue Nov 24, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@Ethspresso
Copy link

Using brush.set() to set the brush to an unknown string value like a brush name with a typo in it, raises TypeError: Cannot read properties of undefined (reading 'param')

Expected result: An error message saying this is not a valid brush name.

Screenshot from 2023-11-24 09-13-16

Tested using the non-minified v1.0.3 in the p5js.org editor.

@acamposuribe acamposuribe added the enhancement New feature or request label Nov 24, 2023
@acamposuribe acamposuribe self-assigned this Nov 24, 2023
@acamposuribe
Copy link
Owner

Hi!
I've included error messages for several functions, but only for the full version
The minified version avoids these messages to compress the code as much as possible

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants