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

Typo on mouseWheel sample code #7613

Open
1 of 17 tasks
swastikCommits opened this issue Mar 9, 2025 · 1 comment · May be fixed by #7614
Open
1 of 17 tasks

Typo on mouseWheel sample code #7613

swastikCommits opened this issue Mar 9, 2025 · 1 comment · May be fixed by #7614
Labels

Comments

@swastikCommits
Copy link

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
  • Internationalization
  • Friendly errors
  • Other (specify if possible)

p5.js version

Latest

Web browser and version

Brave 1.76.73

Operating system

Windows 11 Version 24H2 (Build 26100.3323)

Steps to reproduce this

In the first example of the mouseWheel function, the code is intended to draw a circle in the center of the canvas and increase the circle's size when the user scrolls the mouse wheel. However, the code contains a typo, as it mistakenly changes the circle's X coordinate instead.

Steps:

  1. Go to p5.js.org
  2. Then go to reference and search for mouseWheel()
  3. The first example

Snippet:

//Draw the circle
circle(circleSize, 50, 50);
Copy link

welcome bot commented Mar 9, 2025

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!

@swastikCommits swastikCommits linked a pull request Mar 9, 2025 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant