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

suggestion for changes in createcanvas method's arguments #6551

Closed
1 of 17 tasks
aatiw opened this issue Nov 11, 2023 · 4 comments · Fixed by #6557
Closed
1 of 17 tasks

suggestion for changes in createcanvas method's arguments #6551

aatiw opened this issue Nov 11, 2023 · 4 comments · Fixed by #6557

Comments

@aatiw
Copy link

aatiw commented Nov 11, 2023

Increasing Access

unsure

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)

Feature enhancement details

Refactor the code to use a more descriptive parameter by renaming the arguments and using the rest parameter (...args). This can enhance code readability.
Screenshot 2023-11-11 192808

this issue is similar to #6548 .

Copy link

welcome bot commented Nov 11, 2023

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!

@Jaivignesh-afk
Copy link
Contributor

Can i work on this issue?

@davepagurek
Copy link
Contributor

Hi! Could you clarify how this issue differs from #6548? It sounds like they are both identifying the same problem, are you suggesting a different solution?

@limzykenneth
Copy link
Member

Closing in favour of #6548

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.

4 participants