p5.js icon indicating copy to clipboard operation
p5.js copied to clipboard

orbitControl() customizability

Open eTrey13 opened this issue 1 year ago • 3 comments

Increasing access

Unsure

Most appropriate sub-area of p5.js?

  • [ ] Accessibility
  • [ ] Color
  • [ ] Core/Environment/Rendering
  • [ ] Data
  • [ ] DOM
  • [ ] Events
  • [ ] Image
  • [ ] IO
  • [ ] Math
  • [ ] Typography
  • [ ] Utilities
  • [X] WebGL
  • [ ] Build process
  • [ ] Unit testing
  • [ ] Internationalization
  • [ ] Friendly errors
  • [ ] Other (specify if possible)

Feature enhancement details

Make orbitControl have options to enable/disable if it controls zoom, pan, and rotation. Sometimes only controlling one or two of those would be useful.

eTrey13 avatar Jun 10 '24 17:06 eTrey13

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!

welcome[bot] avatar Jun 10 '24 17:06 welcome[bot]

I am interested in this issue you want a toggle between zoom pan and rotation you can achieve it by making an object having all properties and pass it as argument in function

VANSH3104 avatar Jun 12 '24 10:06 VANSH3104

I am the person involved in modifying orbitControl(). Regarding rotation and zoom, this can be prevented by setting the coefficient to 0. Therefore, I feel like there is little point in making it an option, in my opinion. Regarding pan, it does not work on mobile tphones unless you use two or more fingers. The buttons used on the mouse are also different. I don't quite understand what the problem is.

However, if there is something useful other than the freeRotation option that I have prepared, and it makes sense in terms of convenience, I will not deny its introduction. Of course, in that case, you need to make sure that the default behavior does not change.

inaridarkfox4231 avatar Jun 18 '24 10:06 inaridarkfox4231