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

Add multiline capability to geometry's prompt #257

Closed
wants to merge 3 commits into from
Closed

Add multiline capability to geometry's prompt #257

wants to merge 3 commits into from

Conversation

dinocosta
Copy link

@dinocosta dinocosta commented May 22, 2019

Sometimes I prefer using a multiline prompt, for example when the git branch name is too big. In this situation, I've found myself messing around with geometry in order to use separate the plugins from the user prompt.

As such, this Pull Request adds a PROMPT_GEOMETRY_MULTILINE flag which is set to false by default. Changing that flag to true makes geometry render the prompt in two lines, the first only for the directory and plugins and the second one only for the user prompt.

I've used GEOMETRY_SYMBOL_SPACER as the separator between the directory and the plugins but I don't know if this would be the best option or if it would be better to create a new variable just for this separator so I would appreciate some feedback here.

Also, the contribution guide mentions updating the version but I couldn't find it anywhere except in the changelog. If I forgot to update something feel free to raise that to my attention.

The PROMPT_GEOMETRY_MULTILINE enables having geometry on multiple lines,
rendering the directory and plugins all on the same line and the user
prompt on a newline.
Disable RPROMPT if PROMPT_GEOMETRY_MULTILINE is set to true.
@jedahan
Copy link
Member

jedahan commented Aug 21, 2019

Thanks for making this PR. Unfortunately, we just released v2 of geometry which has a lot of changes to how things work. Would you mind starting a new PR based on the mnml branch? I'd be happy to merge if it was straightforward.

@jedahan jedahan closed this Aug 21, 2019
@dinocosta
Copy link
Author

Hi @jedahan thanks for the reply! 👋
I'll see if I get the time to try and do that :)

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

2 participants