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

Added an option to ignore prefix #7

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

alihammad-gist
Copy link

This pull request adds an option @sessionx-prefix (on by default) which when set to off, will ignore tmux's prefix key when binding a key to launch sessionx popup. The following configuration maps Ctrl+s to launch sessionx popup.

set -g @sessionx-bind 'C-s'
set -g @sessionx-prefix off

@omerxx
Copy link
Owner

omerxx commented Dec 11, 2023

@alihmm @alihammad-gist Cool!
Can you add this to the readme please and I'll merge?
Thanks!

@alihammad-gist
Copy link
Author

@omerxx Added a little description for the option @session-prefix in read me.

@omerxx
Copy link
Owner

omerxx commented Dec 17, 2023

Thanks! Lots of markdown auto fixed it seems like title line breaks and cleaning empty spaces. Are you using a plugin / LSP of some sort for that?

@alihammad-gist
Copy link
Author

Yes, using prettier as a vim filter for formatting. These formatters use some sort of standard formatting so you don't have to memorize what goes where. Prettier is pretty handy cuz it supports a lot of filetypes including markdown. It's available on npm. Btw great plugin, especially the revised design is a lot better.

@omerxx
Copy link
Owner

omerxx commented Jan 3, 2024

@alihammad-gist Gotcha. Care to fix the conflicts so I could merge?

Merge changes in README.md and added a bit to the explaination of
@session-prefix option.
@the-spanish-guy
Copy link

I was looking at this PR and I liked the idea but I saw that it wasn't merged due to the conflicts, is there a plan to fix this?
If necessary, I can open a new pr implementing this

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.

None yet

4 participants