Skip to content

feat(ui): add toggle for bbox with hotkey #8385

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

Conversation

csehatt741
Copy link

Summary

A new feature was implemented to show/hide the bounding box.

  • A method, toggleBboxVisibility, was added to CanvasBboxToolModule, which toggles the visibility flag of the bounding box.
  • A new hotkey hook, useCanvasToggleBboxHotkey, was implemented and integrated into CanvasToolbar. This registers Shift+O as a hotkey in useHotkeyData to trigger the toggleBboxVisibility method, providing a quick keyboard shortcut.
  • A new component, CanvasAlertsBboxVisibility, was introduced, which shows a warning message, when the bounding box is hidden.
  • New translation string were added to en.json for the warning message.

Related Issues / Discussions

Closes #7182

QA Instructions

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • Documentation added / updated (if applicable)
  • Updated What's New copy (if doing a release after this PR)

@github-actions github-actions bot added the frontend PRs that change frontend files label Aug 4, 2025
Copy link
Collaborator

@psychedelicious psychedelicious left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks! One small change requested.

@csehatt741 csehatt741 force-pushed the feature/restore-bbox-hiding-functionality branch from ac3ee6b to 7ef257f Compare August 5, 2025 06:54
Attila Cseh and others added 3 commits August 11, 2025 22:24
If you accidentally hit the hotkey and hide the bbox it could be
difficult to figure out how to un-hide it without the hotkey called out
in the alert.
@psychedelicious psychedelicious force-pushed the feature/restore-bbox-hiding-functionality branch from 7ef257f to cd9f408 Compare August 11, 2025 12:27
Copy link
Collaborator

@psychedelicious psychedelicious left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@psychedelicious psychedelicious enabled auto-merge (rebase) August 11, 2025 12:28
@psychedelicious psychedelicious merged commit 1a11437 into invoke-ai:main Aug 11, 2025
12 checks passed
@csehatt741 csehatt741 deleted the feature/restore-bbox-hiding-functionality branch August 12, 2025 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend PRs that change frontend files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[enhancement]: restore bbox hiding functionality
2 participants