Skip to content

fix: remove duplicate Settings menu item#16

Merged
galz10 merged 3 commits intomainfrom
codex/fix-issue-15-settings-duplicate
Mar 28, 2026
Merged

fix: remove duplicate Settings menu item#16
galz10 merged 3 commits intomainfrom
codex/fix-issue-15-settings-duplicate

Conversation

@galz10
Copy link
Copy Markdown
Owner

@galz10 galz10 commented Mar 26, 2026

Summary

Fixes duplicate Settings... entries in the macOS app menu by removing the extra Settings scene menu contribution and keeping a single settings command wired to IDX0's inline settings UI.

Details

  • Removed the standalone Settings { ... } scene from idx0/App/idx0App.swift.
  • Kept CommandGroup(replacing: .appSettings) with the existing Settings... action and keyboard shortcut (.openSettings).
  • This ensures only one Settings... item appears in the app menu while preserving the existing inline settings flow.

Related Issues

Closes #15

How to Validate

  1. Build the app:
    • xcodebuild -project idx0.xcodeproj -scheme idx0 -destination 'platform=macOS' build
    • Expected: ** BUILD SUCCEEDED **
  2. Launch IDX0.
  3. Open the app menu (IDX0 in the menu bar).
  4. Verify there is only one Settings... item.
  5. Click Settings... and verify the inline settings view opens as before.

Pre-Merge Checklist

  • Builds successfully (xcodebuild / Xcode)
  • Added/updated tests (if needed)
  • Updated relevant documentation and README (if needed)
  • Noted breaking changes (if any)
  • Tested on macOS

@galz10 galz10 merged commit d7c9e58 into main Mar 28, 2026
3 checks passed
@galz10 galz10 deleted the codex/fix-issue-15-settings-duplicate branch March 28, 2026 02:42
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.

[Bug]: Duplicate "Settings..." items in app menu

1 participant