Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add tray icon and set single instance mode to improve startup time on Windows #3324

Merged
merged 1 commit into from
Dec 16, 2018

Conversation

phungtuanhoang1996
Copy link
Contributor

This is my attempt to address ther slow startup time on Windows, and also my first attempt to contribute to an open-source project (because I really like Hyper)

Please feel free to review and comment on what can be improve / went wrong. Thanks! 😄

Changed hyper to be running single instance mode on windows
Added a tray icon with exit option and new window option

Always Preloads a hidden window (without rpc handler and terminal session)
When a new window needs to be opened, the preloaded window will have its rpc and terminal session attached and brough to front

Added a tray icon with exit option and new window option

Always Preloads a hidden window (without rpc handler and terminal session)
When a new window needs to be opened, the preloaded window will have its rpc and terminal session attached and brough to front
@rauchg rauchg requested review from chabou and juancampa December 12, 2018 17:42
@chabou
Copy link
Contributor

chabou commented Dec 12, 2018

🙏 @phungtuanhoang1996 awesome work!

I will review this ASAP.
Thank you so much for your help.

My first thought was: is this a common behavior on Windows to not quit an app? Maybe we should add an opt-out option?
I remember that other apps (like Skype) offer a way to really quit the app (in context menu on tray icon, if I remember correctly).

@phungtuanhoang1996
Copy link
Contributor Author

phungtuanhoang1996 commented Dec 13, 2018

I did include an option to really quit the app in the context menu from the tray icon.
I can make an opt-out option as well.

@chabou
Copy link
Contributor

chabou commented Dec 13, 2018

Oh... I didn't see the systray context menu. Sorry.

For the option, let see if user are complaining about this.
This is the Hyper behavior on macOS and this is totally ok to me.

@chabou chabou changed the title Improve startup time on Windows (addressing #2935 and #2980) Improve startup time on Windows Dec 16, 2018
@chabou chabou changed the title Improve startup time on Windows Add tray icon and set single instance mode to improve startup time on Windows Dec 16, 2018
@chabou chabou merged commit f324a67 into vercel:canary Dec 16, 2018
@chabou
Copy link
Contributor

chabou commented Dec 16, 2018

We are really honored to merge your first open-source contribution 🙏
Thank you!

chabou added a commit that referenced this pull request Dec 22, 2018
chabou added a commit to chabou/hyper that referenced this pull request Dec 22, 2018
juancampa pushed a commit that referenced this pull request Dec 23, 2018
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