Skip to content
New issue

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

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

Already on GitHub? # to your account

docs(readme): reset zsh prompt after calling sesh-sessions script #189

Merged
merged 1 commit into from
Oct 8, 2024
Merged

docs(readme): reset zsh prompt after calling sesh-sessions script #189

merged 1 commit into from
Oct 8, 2024

Conversation

alexandre-abrioux
Copy link
Contributor

@alexandre-abrioux alexandre-abrioux commented Oct 7, 2024

When in a Zsh shell and pressing Alt-s, the session list opens with FZF, thanks to the script provided in the README.

Pressing Esc or Ctrl-C goes back to the shell, but we need to press Enter again before we're given another Zsh prompt. The same issue appears when entering a tmux session: after detaching from tmux, we need to press Enter again before we're given another Zsh prompt.

Calling the function by typing in sesh-session at the Zsh prompt and pressing Enter, and then closing FZF already does what we want - we're given another Zsh prompt. This PR aims to replicate the same behavior through the Alt-s shortcut.

@alexandre-abrioux alexandre-abrioux marked this pull request as ready for review October 7, 2024 20:49
@alexandre-abrioux alexandre-abrioux changed the title docs(readme): improve zsh script docs(readme): reset zsh prompt after calling sesh-sessions script Oct 7, 2024
@alexandre-abrioux alexandre-abrioux marked this pull request as draft October 8, 2024 09:19
@alexandre-abrioux alexandre-abrioux marked this pull request as ready for review October 8, 2024 09:29
@joshmedeski
Copy link
Owner

Thanks @alexandre-abrioux

@joshmedeski joshmedeski merged commit a369e55 into joshmedeski:main Oct 8, 2024
4 checks passed
@alexandre-abrioux alexandre-abrioux deleted the patch-1 branch October 8, 2024 13:21
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants