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

Only save user provided commands in history #59

Merged
merged 2 commits into from
Feb 28, 2024

Conversation

lijenicol
Copy link
Contributor

When using any of the command buttons ( "next", "continue", etc.), those commands were being saved as part of the command history. I found this to be sort of unintuitive, as I would only want the commands I entered in the input field to be part of the history.

Example workflow:

  • I enter in the command pp myVariable in the input field to "pretty print" the value of myVariable
  • I then press the "next" button to go to the next line
  • I then want to hit up on my keyboard to bring the command pp myVariable into the input field. Instead, what happens when I press the up arrow is that the "next line" command is brought in

The changes I make here refactor the code a little such that only commands supplied in the input get saved to the command history.

@romanvm
Copy link
Owner

romanvm commented Nov 29, 2022

Thank you for your contribution. I will check your PR as soon as I can. I apologize in advance if it may take a long time because of my specific circumstances.

@lijenicol
Copy link
Contributor Author

No worries! I'm in no rush

@romanvm romanvm merged commit ccf83ec into romanvm:master Feb 28, 2024
7 checks passed
@romanvm
Copy link
Owner

romanvm commented Feb 28, 2024

I finally found time to review this PR. Thank you for your contribution.

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