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

Close window issue with initialWorkingDirectory #28

Closed
bhardy opened this issue Sep 8, 2017 · 4 comments
Closed

Close window issue with initialWorkingDirectory #28

bhardy opened this issue Sep 8, 2017 · 4 comments
Assignees
Labels

Comments

@bhardy
Copy link

bhardy commented Sep 8, 2017

Hey,

Great plugin. I've been using it for quite a while with no issues and I'm sure it's saved me hours of time.

The other day I was looking through the options and I found initialWorkingDirectory—which is perfect for me, as all of my development work is in in the same base directory. I set it to initialWorkingDirectory: "~/Development", and everything seemed to be working fine until I caught a bug.

Today I was simply trying to close all my Hyper terminals with cmd+w, and that's when I realized I couldn't close the window anymore. All the tabs go away, but I'm left with a little useless
ghost window.

hypercwd

After a little troubleshooting (disabling every other plugin) I realized it was caused entirely by the initialWorkingDirectory option. I've attached a screenshot.

I'm on macOS and using oh-my-zsh.

@bhardy bhardy changed the title Close tab issue with initialWorkingDirectory Close window issue with initialWorkingDirectory Sep 8, 2017
@hharnisc
Copy link
Owner

hharnisc commented Sep 9, 2017

Thanks so much for the report @bhardy - can definitely reproduce this! Digging into it now 😄

@hharnisc
Copy link
Owner

hharnisc commented Sep 9, 2017

Hey @bhardy I figured out what was going on - it was creating a null session and Hyper never ends up destroying it. I submitted a PR to Hyper to fix this, so it might be a bit before it gets deployed to production. More details on that here: vercel/hyper#2191

@bhardy
Copy link
Author

bhardy commented Sep 9, 2017

@hharnisc, right on. Thanks again for the great plugin.

@mhxbe
Copy link

mhxbe commented Sep 20, 2017

It seems that your pull-request already shipped with version 1.4.4 of Hyper.
https://github.com/zeit/hyper/commits/1.4.4

Thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants