Skip to content

✨ Add opt-in radial menu movement across displays - #1141

Open
kasyan1337 wants to merge 1 commit into
mrkai77:developfrom
kasyan1337:feature/radial-menu-across-screens
Open

✨ Add opt-in radial menu movement across displays#1141
kasyan1337 wants to merge 1 commit into
mrkai77:developfrom
kasyan1337:feature/radial-menu-across-screens

Conversation

@kasyan1337

Copy link
Copy Markdown

Description

Adds an opt-in setting, Move radial menu across screens, for multi-display setups.

When enabled, Loop now:

  • Detects when the cursor enters another display while the radial menu is active.
  • Repositions the menu at the cursor on the destination display.
  • Rebases the radial interaction so direction selection starts from the new display.
  • Updates the resize/action target screen to match the cursor destination.
  • Preserves the existing behavior by default and respects the radial-menu center-lock option.

The setting includes a tooltip and localized string entries. The default is off to avoid changing existing users’ behavior.

How has this been tested?

  • xcodebuild build -project Loop.xcodeproj -scheme Loop -configuration Debug -sdk macosx -derivedDataPath /private/tmp/Loop-DerivedData-pr -clonedSourcePackagesDirPath /private/tmp/Loop-Sources CODE_SIGNING_ALLOWED=NO -skipMacroValidation
    • Passed with Xcode 26.6 on macOS.
  • git diff --cached --check
    • Passed before commit.
  • The app was launched locally from Xcode after configuring all three local targets with Sign to Run Locally.
  • The two-physical-display interaction should be acceptance-tested by the maintainer or contributor on a multi-monitor Mac; this environment did not provide a reliable automated two-display fixture.
  • SwiftFormat lint was not run because swiftformat is not installed in the local environment.

The local signing workaround was not included in this PR.

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable
  • I have no unrelated changes in this PR.

AI usage

Codex assisted with implementation, code review, and local build verification. The feature design, repository contribution, and final PR contents were reviewed for this change.

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.

1 participant