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

Honor ZSH_TMUX_ITERM2 for new-session and attach #4422

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shaxbee
Copy link

@shaxbee shaxbee commented Sep 28, 2015

Option ZSH_TMUX_ITERM2 is not taking effect when attaching to or creating new session.
The fix is to scan arguments passed to tmux for new-session and attach and inject '-CC' option when eligible.

@@ -60,8 +60,12 @@ if which tmux &> /dev/null
# Wrapper function for tmux.
function _zsh_tmux_plugin_run()
{
# We attach to existing or start new session
if [[ -n "${@[(r)attach]}" || -n "${@[(r)new-session]}" ]]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might also want to check for the attach-session and new command aliases.

Should this be picking up the "ZSH_TMUX_FIXTERM stuff too, so it matches the attach/new-session behavior of the other branches?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's possible to make this into a one conditional with |:

if [[ -n "${@[(r)attach|attach-session|new|new-session]}" ]]

@mcornella mcornella added the tmux label Dec 2, 2015
@DelightRun
Copy link

DelightRun commented Dec 4, 2016

Yes, this is a bug for all tmux command alias such as ta, ts etc, except tmux command generated by the wrapper function above.
So I think we'd better fix this bug ASAP since this PR has been here for more than 1 year.

@mcornella mcornella added Area: plugin Issue or PR related to a plugin Status: conflicts Pull Request that has conflicts with the master branch labels Mar 24, 2019
@larson-carter
Copy link
Member

@shaxbee Can you please rebase this PR onto our latest master branch. Then update this PR with a Commit please? Thank You!

@praveendhawan
Copy link

Any updates on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: plugin Issue or PR related to a plugin Status: conflicts Pull Request that has conflicts with the master branch
Projects
Status: Backlog
Development

Successfully merging this pull request may close these issues.

None yet

6 participants