Skip to content

Add Configure option to the library context menu - #726

Merged
insidegui merged 1 commit into
insidegui:mainfrom
ElSteverino:feature/library-configure-menu
Aug 3, 2026
Merged

Add Configure option to the library context menu#726
insidegui merged 1 commit into
insidegui:mainfrom
ElSteverino:feature/library-configure-menu

Conversation

@ElSteverino

Copy link
Copy Markdown
Contributor

Motivation

The full VM configuration sheet is currently reachable only via the "Virtual Machine Settings…" button on a VM window's pre-boot screen. That's easy to miss (especially with #725 hiding it below the fold), and it requires opening a session window just to tweak CPU/memory/network settings.

Change

Adds a Configure… item to the library item context menu, right above Duplicate. It presents the existing VMConfigurationSheet in the .postInstall context and saves to the VM bundle via saveMetadata() when the user clicks Done (Cancel writes nothing, mirroring the session sheet's behavior).

The item is disabled while the VM has an open session window: the session owns the model there and auto-saves its own copy of the configuration, so editing from the library at the same time could race it. The existing button on the pre-boot screen covers that case.

Testing

  • Built and ran on macOS 27.0 beta with Xcode 27 beta.
  • Configure… → change CPU/memory → Done → change visible in .vbdata/Config.plist and reflected on next boot; Cancel discards.
  • Item correctly disabled while the VM's window is open, enabled after closing it.
  • Existing validation applies (invalid MAC address blocks saving with the standard error display).

🤖 Generated with Claude Code

The full VM configuration sheet was previously reachable only from the
Virtual Machine Settings button on a VM window's pre-boot screen. Add a
Configure menu item to the library item context menu that presents the
same sheet directly, saving to the VM bundle when the user confirms.

The item is disabled while the VM has an open session window, since the
session owns the model and auto-saves its own copy of the configuration.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@insidegui

Copy link
Copy Markdown
Owner

Been meaning to do this for a while, thanks for investing some tokens into it 😄

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.

2 participants