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

Misleading description of mousePressed() reference #6045

Closed
1 of 17 tasks
yogitheboss opened this issue Mar 4, 2023 · 2 comments · Fixed by #6059
Closed
1 of 17 tasks

Misleading description of mousePressed() reference #6045

yogitheboss opened this issue Mar 4, 2023 · 2 comments · Fixed by #6059

Comments

@yogitheboss
Copy link
Contributor

Most appropriate sub-area of p5.js?

  • Accessibility
  • Color
  • Core/Environment/Rendering
  • Data
  • DOM
  • Events
  • Image
  • IO
  • Math
  • Typography
  • Utilities
  • WebGL
  • Build Process
  • Unit Testing
  • Internalization
  • Friendly Errors
  • Other (specify if possible)

p5.js version

v1.5.0

Web browser and version

108.0.5359.125 (Official Build) (64-bit) (cohort: Stable)

Operating System

Windows

Steps to reproduce this

Steps:

  1. go to the webpage link
  2. the description of the mouse Pressed i.e. Click within the image to change
    the value of the rectangle, is wrong as we don't really need to click within the rectange , clicking anywhere would still change the value.
    also I think we should call the variable color_value instead of value only.

Snippet:

image

@yogitheboss
Copy link
Contributor Author

Can I work on this issue? @davepagurek @limzykenneth

@davepagurek
Copy link
Contributor

Thanks @yogitheboss, I'll assign this to you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants