Skip to content

Conversation

@mrymtsk
Copy link
Contributor

@mrymtsk mrymtsk commented Feb 25, 2020

In prior to #19, #{prefix_highlight} was replaced with empty prompt or prefix prompt or copy prompt. However, #19 changed the behavior so that (empty prompt or prefix prompt) and (copy prompt or sync prompt or none) are displayed.
This PR restores the behavior to show only one of the four possible prompts: prefix, copy, sync, and empty (prioritized in this order).
In addition to this change, , and } in prompts and attribute strings are now escaped as #, and #} to simplify the script. Now ${prefix,copy,sync,empty}_highlight are included in ${prefix,copy,sync,empty}_with_optional_affixes.

@erickpintor
Copy link
Collaborator

I'm so sorry for the delay. That's a great contribution.

Could you please rebase you branch with master? We can merge it afterwards.

@mrymtsk
Copy link
Contributor Author

mrymtsk commented Mar 23, 2020

Thank you for the reply and I rebased the branch with master.

@erickpintor erickpintor merged commit 1db6e73 into tmux-plugins:master Mar 25, 2020
@erickpintor
Copy link
Collaborator

Thank you!

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.

2 participants