Skip to content

feat(cardinal):Bring window to focus on the active Space(#188)#209

Open
xiaxiansong wants to merge 2 commits into
cardisoft:masterfrom
xiaxiansong:feature/activate-window-on-current-space
Open

feat(cardinal):Bring window to focus on the active Space(#188)#209
xiaxiansong wants to merge 2 commits into
cardisoft:masterfrom
xiaxiansong:feature/activate-window-on-current-space

Conversation

@xiaxiansong

@xiaxiansong xiaxiansong commented May 28, 2026

Copy link
Copy Markdown

在哪个桌面触发唤醒,Cardinal 的窗口就移动到那个 Space 并显示出来,用快捷键唤出 cardinal 窗口时不会再回到启动桌面

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds macOS-specific window activation behavior so Cardinal’s main window moves to the currently active Space before being shown/focused.

Changes:

  • Refactors window activation through a platform-specific wrapper.
  • On macOS, schedules activation on the main thread and updates NSWindowCollectionBehavior.
  • Adds macOS-only unit tests for the collection behavior flag transformation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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