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

vi mode indicator #139

Closed
codesections opened this issue Feb 6, 2019 · 1 comment
Closed

vi mode indicator #139

codesections opened this issue Feb 6, 2019 · 1 comment
Labels
🙏 help wanted needs PR or help on decision ⭐️ idea feature request or suggestion
Milestone

Comments

@codesections
Copy link
Contributor

Thanks for this project—I just switched from Zsh to Fish, and I'm not sure I would have done so if it meant needing to give up the wonderful .

However, I also use the vim keybindings in Fish, and I noticed that this project indicates the current mode a bit differently than the Zsh Pure does. Specifically, our version prints out the full default [N] or other letter to indicate the current mode. In contrast, the Zsh version flips the to a for every non-insert mode.

IMO, this looks a lot better, fits in with the rest of the prompt, and doesn't sacrifice any useful info. I've already modified my version of the prompt to duplicate this behavior, and would be happy to submit a PR upstream if y'all would be interested in it.

If so, should it be default behavior or user-configurable? It's tempting to make everything user-configurable, but it can also be overwhelming to expose too many knobs and dials.

@edouard-lopez edouard-lopez added the ⭐️ idea feature request or suggestion label Feb 7, 2019
@edouard-lopez
Copy link
Member

Hello @codesections,
Glad we ease the transition to fish ✌️


Disclaimer: I'm not a vi-mode user myself.

Having similar behaviour than the zsh version is something we aim for as we recently merged several PRs to fill the gap between our version and the one from sindresorhus.

So my recommendation would be to set the behavior you described as the default and add a flag to switch back to the fish default behavior, e.g.:

pure_reverse_prompt_symbol_in_vimode true

We are currently relying on the default behavior of fish by calling fish_default_mode_prompt in _pure_prompt_vimode.fish:
https://github.com/rafaelrinaldi/pure/blob/1fabe247ce13a1cb4d3de3a219f0116f6af3bb43/functions/_pure_prompt_vimode.fish#L2

ℹ️ Have a look at our configuration revamp and contributing guide to get familiar with naming convention.

cc @schrodincat @rafaelrinaldi

@edouard-lopez edouard-lopez modified the milestone: v2.x-future Feb 7, 2019
@edouard-lopez edouard-lopez added the 🙏 help wanted needs PR or help on decision label Feb 7, 2019
@edouard-lopez edouard-lopez added this to the v2.0.0-next milestone Feb 18, 2019
edouard-lopez added a commit to codesections-forks/pure that referenced this issue Feb 19, 2019
edouard-lopez added a commit that referenced this issue Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🙏 help wanted needs PR or help on decision ⭐️ idea feature request or suggestion
Projects
None yet
Development

No branches or pull requests

2 participants