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

GitButler will not close (CMD+Q) when error pip-ups are present #5763

Open
krstp opened this issue Dec 6, 2024 · 6 comments
Open

GitButler will not close (CMD+Q) when error pip-ups are present #5763

krstp opened this issue Dec 6, 2024 · 6 comments
Labels
bug Something isn't working 🎉reproduced🎉 The issue could be reproduced by following the instructions

Comments

@krstp
Copy link

krstp commented Dec 6, 2024

Version

0.14.2

Operating System

macOS

Distribution Method

dmg (Apple Silicon)

Describe the issue

Unable to quit app when error pop-ups are floating in the bottom-right corner. Had to dismiss all, only then I was able to close the app.

How to reproduce

Cause an error message. Try to CMD+Q app.

Expected behavior

App should close gracefully terminating any active processes or possibly ask to verify the quit process if there are other active processes.

Relevant log output

No response

@krstp krstp added the bug Something isn't working label Dec 6, 2024
@Byron
Copy link
Collaborator

Byron commented Dec 7, 2024

Thanks a lot for reporting!

Could you provide more information, as maybe the type of error matters as well.

I have recorded a video where Cmd + Q works fine even with an error popup visible.
https://github.com/user-attachments/assets/0519470d-f8cf-4993-acf8-77b673f41870

Thanks for your help!

@Byron Byron added the feedback requested Feedback was requested to help resolve the issue label Dec 7, 2024
@krstp
Copy link
Author

krstp commented Dec 8, 2024

@Byron Literally, try drag'n'drop a hunk from one commit to the other, similar to here (#5761 ; about to upload a video).

After error shows up, CMD+Q will not activate. No point of making video as the UI will not react to CMD+Q.

I hope this helps.

Screenshot 2024-12-07 at 11 31 41 PM

@krstp
Copy link
Author

krstp commented Dec 8, 2024

BTW: What a bizarre thing, now it works :| Second ago I did try it and it was resulting with exact error. I am baffled.

@Byron Byron added 🎉reproduced🎉 The issue could be reproduced by following the instructions and removed feedback requested Feedback was requested to help resolve the issue labels Dec 8, 2024
@Byron
Copy link
Collaborator

Byron commented Dec 8, 2024

Turns out that sometimes quitting works, but sometimes it doesn't - I even have it on video but chose to only upload the part where the popup blocks quitting.

Untitled.mov

Not shown in the video is how closing the popup makes quitting possible again.

CC @estib-vega - maybe it's something about hotkey capturing in error popups?

@estib-vega
Copy link
Contributor

@krstp Thanks for reporting and @Byron thanks for tagging me!

We've observed this behavior in the wild, but every time we start chasing it, we can't seem to reproduce.
I'm glad you two seem to have found a reproducible condition.
I'll take a look!

@estib-vega
Copy link
Contributor

I think I have a potential fix!
#5785

Thanks again for the reproducible example!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 🎉reproduced🎉 The issue could be reproduced by following the instructions
Projects
None yet
Development

No branches or pull requests

3 participants