Skip to content

Additional messages for gizmos - #745

Merged
lawsie merged 1 commit into
flipcomputing:mainfrom
lawsie:gizmo-usability-minor-fix
Aug 11, 2026
Merged

Additional messages for gizmos#745
lawsie merged 1 commit into
flipcomputing:mainfrom
lawsie:gizmo-usability-minor-fix

Conversation

@lawsie

@lawsie lawsie commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Show a message to surface the fact that the keyboard cursor can be triggered by using arrow keys
image
  • Show a message to surface the fact that the on-screen controls can be hidden/shown with the cog button
image

Translate for locales.

AI usage

Claude Sonnet 5 wrote the translations and added the messages in the right place.

Summary by CodeRabbit

  • New Features

    • Added localized guidance for selecting objects with the keyboard across supported languages.
    • Added translated instructions explaining how to show or hide gizmo controls.
    • Scale, rotation, and position tools now display helpful controls guidance when keyboard interaction begins.
    • Selection workflows provide clearer prompts when an object must be chosen.
  • Bug Fixes

    • Gizmo hints are now cleared when gizmo interactions end, preventing outdated messages from remaining visible.

@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 84df3e9d-70df-4417-9dea-406b0dbef75d

📥 Commits

Reviewing files that changed from the base of the PR and between 773ad03 and 48003d4.

📒 Files selected for processing (9)
  • locale/de.js
  • locale/en.js
  • locale/es.js
  • locale/fr.js
  • locale/it.js
  • locale/pl.js
  • locale/pt.js
  • locale/sv.js
  • ui/gizmos.js

📝 Walkthrough

Walkthrough

Adds localized mesh-selection prompts and gizmo-control hints in eight locales. Gizmo flows display hints during scale, rotation, and position interactions, pass translated prompts to mesh picking, clear hints on exit, and use the prompt in selection mode.

Changes

Gizmo localization

Layer / File(s) Summary
Localized interaction messages
locale/de.js, locale/en.js, locale/es.js, locale/fr.js, locale/it.js, locale/pl.js, locale/pt.js, locale/sv.js
Each locale adds select_mesh_prompt and gizmo_controls_hint.
Gizmo prompt and hint wiring
ui/gizmos.js
Gizmo activation and mesh-picking flows display translated messages. Gizmo cleanup clears the controls hint. Selection mode passes the translated mesh-selection prompt.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Suggested reviewers: tracygardner, claude

🚥 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 accurately identifies the main change: adding user-facing messages for gizmos.
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

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

@lawsie
lawsie merged commit 3357d30 into flipcomputing:main Aug 11, 2026
3 checks passed
@lawsie
lawsie deleted the gizmo-usability-minor-fix branch August 11, 2026 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant