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

Feature request: Option to remove push button #8

Open
surjikal opened this issue Dec 16, 2024 · 1 comment
Open

Feature request: Option to remove push button #8

surjikal opened this issue Dec 16, 2024 · 1 comment

Comments

@surjikal
Copy link

surjikal commented Dec 16, 2024

There's a (closed) PR about hiding the "Push" button, in the old repo. It's a neat feature, I've been using it for a year or two.

Here's why @MircoBabin created this patch:

Because the "push" button is so close to the "commit" button, I frequently find myself pushing the wrong button. Of course the "push" button displays a window asking for confirmation. And I can cancel that then. But on a bad day in a hurry, I don't read and just push go ahead. Resulting in the merge commit I did not want.


Here's the PR:
prati0100/git-gui#74

Considering merging it, if you like!

I found it useful and figured it might be worth bringing it up here, just in case others find it useful, even if it doesn't get merged.

@j6t
Copy link
Owner

j6t commented Dec 16, 2024

I don't feel very compelled to have this patch. If the only reason is that you push it accidentally and only rarely so, then the subsequent dialog is sufficient safety net that nothing bad happens. The commit message isn't erased, either, so nothing is lost.

BTW, I suggest you get used to using hotkeys. That for Commit is Ctrl-Return.

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

No branches or pull requests

2 participants