Skip to content

video: add a shortcut and menu item to hide the status bar#241

Merged
LinuxJedi merged 1 commit into
LinuxJedi:mainfrom
hobbo91:feature/hide-status-bar
Jul 20, 2026
Merged

video: add a shortcut and menu item to hide the status bar#241
LinuxJedi merged 1 commit into
LinuxJedi:mainfrom
hobbo91:feature/hide-status-bar

Conversation

@hobbo91

@hobbo91 hobbo91 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Cmd+Shift+F (Alt+Shift+F on Windows/Linux) now toggles the status bar at the bottom of the window, complementing the existing fullscreen shortcut. When the bar is hidden the emulated display scales up to fill the space it occupied while keeping its aspect ratio — the canvas simply loses the bar's height, the same kind of change as switching pixel aspect, so the texture and window are resized the same way.

It's also available as a Status Bar item in the hamburger menu next to Fullscreen, and it's listed in the keyboard-shortcuts panel and the UI guide. Hiding shows a brief on-screen message ("Status bar hidden (Cmd+Shift+F restores)"), and restoring shows "Status bar restored", so the shortcut is discoverable even if you toggled it by accident.

Status-bar hit-testing self-gates when the bar is hidden (its cursor region collapses to nothing), and the recorder only ever captures the display region, so recordings are unaffected.

Screenshot 2026-07-20 at 21 51 29 Screenshot 2026-07-20 at 21 51 35 image

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a user-facing toggle to show/hide the bottom status bar in the video frontend, exposed via both a host shortcut and the hamburger menu, with supporting UI/docs updates.

Changes:

  • Add Cmd+Shift+F / Alt+Shift+F shortcut and a hamburger-menu item to toggle the status bar.
  • Make window/texture sizing account for the status bar being hidden, and suppress status-bar drawing while hidden.
  • Update shortcuts panel and user guide documentation to list/describe the new toggle.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
src/video/window.rs Implements status-bar toggle behavior, including shortcut handling, render sizing, and conditional status-bar drawing.
src/video/ui.rs Adds the new menu item, label state, shortcuts-panel row, and tests for menu labeling.
src/video/mod.rs Introduces a global atomic flag to track whether the status bar is hidden.
docs/guide/ui.md Documents the new shortcut and menu behavior in the UI guide.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/video/window.rs
Comment thread src/video/ui.rs
Cmd/Alt+Shift+F now shows or hides the status bar, complementing the fullscreen
shortcut. Hidden, the emulated display scales up to fill the space the bar
occupied, keeping its aspect: the canvas loses the bar's height, the same
change as a pixel-aspect switch, so it resizes the texture and window the same
way. The status-bar hit-testing self-gates (its cursor region collapses to
nothing), and the recorder captures only the display region, so a recording is
unaffected.

Also available as a Status Bar menu item next to Fullscreen, listed in the
keyboard-shortcuts panel and the UI guide.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@hobbo91
hobbo91 force-pushed the feature/hide-status-bar branch from 3ce2561 to 9a9dd1b Compare July 20, 2026 21:30

@LinuxJedi LinuxJedi left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Good call. I didn’t think of that.

@LinuxJedi
LinuxJedi merged commit 2538648 into LinuxJedi:main Jul 20, 2026
14 checks passed
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.

3 participants