Bugfix: Gizmo button helpers

[lawsie]

Summary

Don't show gizmo button helper numbers when you aren't using keyboard controls.

AI usage

Claude Sonnet 5 did this.

Summary by CodeRabbit

• Bug Fixes
  • Improved keyboard-based object placement by ensuring selections only target valid, pickable items.
  • Fixed placement behavior so selected objects are positioned accurately based on the canvas location.
  • Prevented the gizmo overlay menu from appearing unexpectedly during canvas picking and keyboard navigation.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 3902212a-bfbb-4d84-84a3-f7687ec02f2a

📥 Commits

Reviewing files that changed from the base of the PR and between bf321cd and ac29893.

📒 Files selected for processing (2)

• ui/addmenu.js
• ui/gizmos.js

💤 Files with no reviewable changes (2)

• ui/addmenu.js
• ui/gizmos.js

---

📝 Walkthrough

Walkthrough

Keyboard placement now validates hits against pickable meshes and forwards validated canvas coordinates to the active pick handler. Canvas picking no longer automatically enables the Gizmo menu.

Changes

Placement picking behavior

Layer / File(s)	Summary
Keyboard placement and canvas picking ui/addmenu.js, ui/gizmos.js	Keyboard placement supplies pick validation and keyboard-navigation state to canvas keyboard mode, forwards converted coordinates to the active pick handler, and no longer enables the Gizmo menu during canvas picking.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

• flipcomputing/flock#539: Updates the same placement keyboard flow with canvas picking and an isValidHit predicate.
• flipcomputing/flock#694: Connects Gizmo menu hooks with placement and cancellation behavior.
• flipcomputing/flock#708: Modifies startPlacementKeyboardMode and Gizmo overlay toggling behavior.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title is concise and correctly describes the gizmo helper bugfix reflected by the keyboard/picking changes.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[lawsie]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.