Feature: Allow orbit in sketch mode via setting

[nadr0]

closes #4441

Issue

User wants the ability to rotate while in sketch mode

Implementation

• Used camControls.enableRotate to allow disabling and enabling of the camera orbit while in sketch mode
• Added a setting specifically for allowing orbit in sketch mode
• When the user disables the orbit mode while in sketch mode it will sync their camera to the engine and lock their orbit

Testing

Tested several different workflows

• Refresh app with setting on -> try sketching -> can rotate
• Refresh app with setting off -> try sketching -> cannot rotate
• Refresh app with setting on -> try sketching -> turn it off -> camera should sync and cannot rotate
• Refresh app with setting off -> try sketching -> turn it on -> can rotate

[qa-wolf]

QA Wolf here! As you write new code it's important that your test coverage is keeping up.
Click here to request test coverage for this PR!

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
modeling-app	✅ Ready (Inspect)	Visit Preview	Jan 16, 2025 3:27pm

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.99%. Comparing base (d4e9552) to head (47a766e).
Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4990   +/-   ##
=======================================
  Coverage   85.99%   85.99%           
=======================================
  Files          88       88           
  Lines       31470    31475    +5     
=======================================
+ Hits        27062    27067    +5     
  Misses       4408     4408           

Flag	Coverage Δ
wasm-lib	85.99% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pierremtb]

Oh this is cool! Is it ready to review and merge @nadr0 ?

[nadr0]

Oh this is cool! Is it ready to review and merge

Yes this should be ready for review.

[pierremtb]

Looks good to me, tested locally too 🔥 🚀

[pierremtb]

@nadr0 I think this test failure is real