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 option to enable disconnect notifications #39

Closed
MisterTea opened this issue Jun 13, 2017 · 9 comments
Closed

Add option to enable disconnect notifications #39

MisterTea opened this issue Jun 13, 2017 · 9 comments

Comments

@MisterTea
Copy link
Owner

ET used to notify people on disconnect but this was disabled since it caused issues with "tmux -CC". This issue is to add that back as an option for people who do not use tmux control mode

@MisterTea
Copy link
Owner Author

I tried to revive 118210c but it doesn't work well in many cases. Basically, this is very hard to do without a deep integration with a terminal emulator. I could pass some escape sequence letting the terminal emulator know that there is a disconnect, but they would have to render something and handle removing it after we reconnect.

@MisterTea
Copy link
Owner Author

Actually maybe I can change the title of the window. Let me look into that

@MisterTea
Copy link
Owner Author

Actually it's non-trivial to change the window title. I tried setting PROMPT_COMMAND but that doesn't seem to be portable :-/

@krayon
Copy link

krayon commented Jul 7, 2017

Perhaps you could do it for users of tmux at least, by manipulating the status bar(s)? In the event control mode allows for setting stuff like that, you could probably flash a colour or similar? Seems there's a fair bit of customisation available:
https://leanpub.com/the-tao-of-tmux/read#status-bar

As long as you can retrieve current settings, you could change it, then change it back after a delay. You could even have a place holder for an indicator (such as "%%ETDCNT") that could contain the number of disconnections etc.

I am not sure how feasible all this is, just a thought.

@MisterTea
Copy link
Owner Author

@krayon This is a great idea but I don't have time at the moment. Would this be something that you would be interested in submitting a PR for? If you made a standalone C++ class with a connect() disconnect() API I can write the logic to call it.

@ddickstein
Copy link

ddickstein commented Oct 3, 2017

Any further plans here? This would be really helpful, especially when trying to differentiate cases of disconnection vs frozen programs at a glance. (The tmux approach proposed would work for me)

@MisterTea
Copy link
Owner Author

@ddickstein Care to submit a PR? :-)

I'm a tmux -CC user so this isn't high pri for me TBQH but happy to review a PR for it.

@ddickstein
Copy link

What changes do you think are necessary to get it to work? I haven't spent much time working in C++ and I don't know the codebase here. Do you have a rough sense of how much work it is and what the steps would be?

@krayon
Copy link

krayon commented Oct 7, 2017

@MisterTea I absolutely would but unfortunately I don't have the time and I don't actually use EternalTCP myself. I was just evaluating it for someone and had that idea. I'm sorry I can't offer a PR for this :(

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

No branches or pull requests

3 participants