fix(ui): load previously selected theme on startup

[davo0411]

fixes an oversight mentioned in #1648

untested but logically should just work

Summary by CodeRabbit

• Bug Fixes
  • Enhanced theme restoration on app launch with improved fallback support to prevent theme loading failures.

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

[coderabbitai]

📝 Walkthrough

Walkthrough

The Menu::Load function was enhanced to handle theme restoration for returning users. When a previously selected custom theme preset is saved, the code now attempts to load it and falls back to the "Default" preset if loading fails, with appropriate logging.

Changes

Cohort / File(s)	Summary
Theme Loading Fallback src/Menu.cpp	Added conditional logic in Menu::Load to load saved theme presets for non-first-launch scenarios; includes warning log on load failure and automatic fallback to "Default" preset with SelectedThemePreset update.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A theme once chosen, now restored,
When themes take flight, we've got a cord—
Default awaits, a safety's call,
Fallback paths catch one and all! 🎨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: loading a previously selected theme on startup, which directly matches the code modifications in Menu.cpp.

✨ Finishing touches

• 📝 Generate docstrings

---

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: 5ed6701
Using base ref: 5ed6701
Base commit date: 2025-12-31T14:40:51+10:00 (Wednesday, December 31, 2025 02:40 PM)
No actionable suggestions for changed features.

[coderabbitai]

Actionable comments posted: 1

🧹 Nitpick comments (1)

src/Menu.cpp (1)

238-247: Suggest adding issue reference keyword to PR description.

Per the project's coding guidelines, consider updating the PR description to use a GitHub keyword for automatic issue linking:

• Change "Addresses an oversight referenced in issue #1648" to "Fixes #1648" or "Closes #1648"

This will automatically close the issue when the PR is merged and creates a clearer link between the fix and the reported problem.

Based on coding guidelines, "When reviewing PRs, please provide suggestions for Issue References (if PR fixes bugs or implements features): Suggest adding appropriate GitHub keywords: 'Fixes #123' or 'Closes #123' for bug fixes."

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1bb4d08 and 6c54a6c.

📒 Files selected for processing (1)

• src/Menu.cpp

🧰 Additional context used

📓 Path-based instructions (3)

**/*.{cpp,cxx,cc,c,h,hpp,hxx,hlsl,hlsli,fx,fxh,py}

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

Do not include TODO/FIXME placeholders; provide complete, working solutions

Files:

• src/Menu.cpp

src/**/*.{cpp,cxx,cc,h,hpp,hxx}

📄 CodeRabbit inference engine (.github/copilot-instructions.md)

src/**/*.{cpp,cxx,cc,h,hpp,hxx}: Ensure SE/AE/VR runtime compatibility; use runtime detection patterns (e.g., REL::RelocateMember())
Include robust error handling and resource management with graceful degradation in the plugin code

Files:

• src/Menu.cpp

**/*

⚙️ CodeRabbit configuration file

**/*: When reviewing PRs, please provide suggestions for:

1. Conventional Commit Titles (if not following https://www.conventionalcommits.org/ or
   if the existing title does not describe the code changes):
   Format: type(scope): description
   Length: 50 characters limit for title, 72 for body
   Style: lowercase description, no ending period
   Examples:

   • feat(vr): add cross-eye sampling
   • fix(water): resolve flowmap bug
   • docs: update shader documentation

2. Issue References (if PR fixes bugs or implements features):
   Suggest adding appropriate GitHub keywords:

   • "Fixes #123" or "Closes #123" for bug fixes
   • "Implements #123" or "Addresses #123" for features
   • "Related to #123" for partial implementations

Otherwise, use your standard review approach focusing on code quality.

Files:

• src/Menu.cpp

[github-actions]

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