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

Risky enter confirmation #3471

Open
undg opened this issue Apr 3, 2024 · 3 comments
Open

Risky enter confirmation #3471

undg opened this issue Apr 3, 2024 · 3 comments

Comments

@undg
Copy link
Contributor

undg commented Apr 3, 2024

Topic
lazygit can do a many irreversible changes in git repo (local and remote). For those changes we have extra confirmation step with popup widow what is rely good idea. What is not so great and danger, is that confirmation is on <CR>. It would be nice to rebind it to y/n or at least have an option to do so in config.

Your thoughts
Lets take example that makes me a bit anxious every time I'm doing it with lazygit.

  1. Go to Branch window, select branch
  2. d to delete
  3. In option window arrow down on "Delete remote branch" or stay on "Delete local branch"
  4. enter followed by second enter to delete and confirm

This is common workflow for many actions with confirmation.

@undg undg added the discussion label Apr 3, 2024
@mark2185
Copy link
Collaborator

mark2185 commented Apr 3, 2024

There's a similar discussion in this thread.

@undg
Copy link
Contributor Author

undg commented Apr 3, 2024

Also similar discussion here #3300 , though they are are about specific use cases, I'm more about general approach.

@AndrewSav
Copy link
Contributor

#2009

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants