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 back multiline support #260

Closed
AlexanderThaller opened this issue Mar 14, 2020 · 5 comments · Fixed by #262
Closed

Add back multiline support #260

AlexanderThaller opened this issue Mar 14, 2020 · 5 comments · Fixed by #262
Labels
new feature New feature or request

Comments

@AlexanderThaller
Copy link

It seems to be that multi-line command where supported in an older version (#88).

Is that a feature that will be coming back? I can also try to implement it and open a pull request but want to make sure that this is a feature that is actually wanted first.

Thanks for all your work its a nice tool :D

@AlexanderThaller AlexanderThaller added the new feature New feature or request label Mar 14, 2020
@welcome
Copy link

welcome bot commented Mar 14, 2020

Thanks for opening your first issue here! In case you're facing a bug, please update navi to the latest version first. Maybe the bug is already solved! :)

@AlexanderThaller
Copy link
Author

Right now multiline commands look like this which I dont think is super usefull:

I'm not totally sure how to display multiline commands yet.
Maybe displaying the newlines as \n in the command would be good enough for now. I think making sure that the newlines are put back when the command is printed/executed would be a good idea so that snippets that rely on newlines to be correct are still working.

@denisidoro
Copy link
Owner

You can write multi-line commands using \

Could you share your .cheat?

@denisidoro
Copy link
Owner

I'm working on a PR that doesn't require \

@AlexanderThaller
Copy link
Author

Thank you for the quick response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants