Skip to content

fix: open the running app instead of starting another copy - #105

Open
TNguyen1228 wants to merge 1 commit into
microsoft:mainfrom
TNguyen1228:fix/single-instance
Open

TNguyen1228 wants to merge 1 commit into
microsoft:mainfrom
TNguyen1228:fix/single-instance

Conversation

@TNguyen1228

Copy link
Copy Markdown

Each launch from the desktop or Start menu shortcut started a full second Skill Recorder: another main process with its own tray icon, windows and recorder, while the global shortcut could only belong to the first one.

Take Electron's single-instance lock at startup. A later launch exits before creating any window, and the running copy comes to the front on "second-instance", the same way a Dock click does on macOS: the recording bar while recording, otherwise the recorder window (restored if minimized).

Note: when a new revision is installed while an older one is still open, the new launch now hands over to the open copy, so quit it first.

Each launch from the desktop or Start menu shortcut started a full second
Skill Recorder: another main process with its own tray icon, windows and
recorder, while the global shortcut could only belong to the first one.

Take Electron's single-instance lock at startup. A later launch exits before
creating any window, and the running copy comes to the front on
"second-instance", the same way a Dock click does on macOS: the recording bar
while recording, otherwise the recorder window (restored if minimized).

Note: when a new revision is installed while an older one is still open, the
new launch now hands over to the open copy, so quit it first.
@TNguyen1228

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

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