feat(UI): add auto-hide featurelist option

[alandtse]

Auto hide the feature list. Find under General -> Interface -> Behavior.

Summary by CodeRabbit

• New Features
  • Added auto-hide for the left feature list panel with hover-to-reveal behavior.
  • Added a new "Behavior" settings tab where auto-hide options can be configured.
  • Auto-hide preference is now persisted in the app settings.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

Adds an AutoHideFeatureList setting plus theme constants, implements hover-and-delay logic to auto-show/hide the left feature list during rendering, and moves styling/behavior controls into a new Behavior settings tab.

Changes

Cohort / File(s)	Summary
Configuration & Serialization src/Menu.h, src/Menu.cpp	Added bool AutoHideFeatureList to Menu::Settings and included it in JSON (Nlohmann) serialization.
Theme/Layout Constants src/Menu/ThemeManager.h	Introduced three constants: AUTOHIDE_ACTIVATION_ZONE_WIDTH, AUTOHIDE_EXPAND_DELAY, AUTOHIDE_PANEL_WIDTH_RATIO to drive auto-hide behavior and layout.
Feature List Rendering Logic src/Menu/FeatureListRenderer.cpp	Added internal ShouldShowLeftPanel() to track hover state/timing and updated RenderFeatureList() to conditionally render either two-column (left list + right config) or single-column layout based on visibility.
Settings UI & Tabs src/Menu/SettingsTabRenderer.h, src/Menu/SettingsTabRenderer.cpp	Declared and implemented SettingsTabRenderer::RenderBehaviorTab() and integrated it into the Interface tab flow; moved prior styling/behavior controls into this new tab.

Sequence Diagram

sequenceDiagram
    participant User
    participant Renderer as FeatureListRenderer
    participant Helper as ShouldShowLeftPanel()
    participant Theme as ThemeManager::Constants
    participant UI as UI Layout Renderer

    User->>Renderer: Frame render / mouse events
    Renderer->>Helper: Query visibility (AutoHideFeatureList, mouse pos, time)
    Helper->>Theme: Read activation zone width, delay, panel width ratio
    alt AutoHideFeatureList enabled
        Helper->>Helper: Track hover start / evaluate delay
        alt Hover active >= delay or cursor over panel
            Helper-->>Renderer: visible = true
        else
            Helper-->>Renderer: visible = false
        end
    else
        Helper-->>Renderer: visible = true
    end
    Renderer->>UI: Render either two-column or single-column layout based on visible
    UI-->>User: Composited frame

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

• refactor: break apart menu #1330: Modifies the same menu components and UI layout foundations which this change builds upon.
• refactor(ui): improve menu layout and organization #1178: Related menu/feature-list layout changes and feature-detail UI refactoring.
• feat(ui): theme management fixes and enhancements #1700: Overlaps with SettingsTabRenderer UI organization changes.

Suggested reviewers

• doodlum
• davo0411
• jiayev

Poem

🐰
Soft paws at the screen, I peep and peep,
A hidden list wakes from its sleep,
Hover, wait — it blooms just so,
Two columns dance, then gently go,
Hooray for tidy menus — hop, hop, we leap!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 18.18% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title 'feat(UI): add auto-hide featurelist option' directly and accurately summarizes the main change: adding a new auto-hide feature list functionality to the UI.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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 and usage tips.}

[github-actions]

Using provided base ref: f02aad7
Using base ref: f02aad7
Base commit date: 2026-01-31T16:30:08-08:00 (Saturday, January 31, 2026 04:30 PM)
No actionable suggestions for changed features.

[Copilot]

Pull request overview

Adds an “Auto-hide Feature List” UI option and supporting logic to dynamically hide/show the left feature panel based on cursor position, exposed under the new Interface → Behavior sub-tab.

Changes:

• Introduces a new Interface → Behavior sub-tab and moves several UI behavior-related settings into it (including the new auto-hide toggle).
• Adds auto-hide visibility logic to the feature list renderer to collapse the left panel and expand it on hover with a delay.
• Adds auto-hide tuning constants to ThemeManager::Constants and a new Menu::Settings flag.

Reviewed changes

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

Show a summary per file

File	Description
src/Menu/ThemeManager.h	Adds constants used to control auto-hide activation zone/delay/width heuristics.
src/Menu/SettingsTabRenderer.h	Declares the new RenderBehaviorTab() sub-tab.
src/Menu/SettingsTabRenderer.cpp	Implements the Behavior tab UI and adds the Auto-hide Feature List toggle.
src/Menu/FeatureListRenderer.cpp	Implements runtime show/hide behavior for the left panel and switches table layout between 1/2 columns.
src/Menu.h	Adds AutoHideFeatureList to the menu settings struct.

[coderabbitai]

Actionable comments posted: 2

🤖 Fix all issues with AI agents

In `@src/Menu.h`:
- Line 382: AutoHideFeatureList (a bool field) is deserialized via the broad
settings = o_json in Menu::Load(), which can silently accept malformed
non-boolean JSON values; add explicit type validation when reading boolean
settings (including AutoHideFeatureList) from o_json: check that the JSON entry
exists and is_boolean() before assigning, otherwise fall back to the default
false or log a warning/error. Update the deserialization code path that uses
settings = o_json (and any subsequent direct reads of AutoHideFeatureList) to
perform these checks to ensure robust handling of malformed config values.

In `@src/Menu/FeatureListRenderer.cpp`:
- Around line 32-81: The auto-hide logic in ShouldShowLeftPanel only considers
mouse position and blocks keyboard/gamepad/VR users; update ShouldShowLeftPanel
to force visibility when ImGui navigation or VR is active by checking
ImGui::GetIO().NavActive (or appropriate ImGui::GetIO() nav flag) and a VR state
(e.g., globals::menu->IsVRMode() or a VR flag in settings) alongside the
existing mouse checks; if nav or VR is active set leftPanelVisible = true and
return true early so keyboard/gamepad/VR navigation can reveal the panel even
when auto-hide is enabled.

🧹 Nitpick comments (1)

src/Menu/ThemeManager.h (1)

188-191: Consider scaling the auto-hide constants by effective UI scale.

A fixed 10px activation zone can be hard to hit on high DPI or VR UI scales. Using the effective UI scale (e.g., exp2(GlobalScale)) to scale the activation width/delay would keep the interaction consistent across SE/AE/VR setups.

[github-actions]

✅ A pre-release build is available for this PR:
Download

[davo0411]

Does this behave like the windows taskbar popping up/down based on mouse position? if so very cool

[davo0411]

do we need a new tab? this essentially replaces 99% of the styling tab. atp just combine them or use the old styling tab

[doodlum]

Presumably alandtse assumes this will be used for other things in the future

[alandtse]

It moves some of the existing options in that are more behavior and keeps styling to styling.