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

Deleting local branch doesn't update the UI #978

Closed
s101d1 opened this issue Jul 31, 2019 · 7 comments
Closed

Deleting local branch doesn't update the UI #978

s101d1 opened this issue Jul 31, 2019 · 7 comments

Comments

@s101d1
Copy link

s101d1 commented Jul 31, 2019

I'm on Git Cola v3.4 in manjaro linux.

How to produce the issue:

  • In the Branches view, delete one of the local branch item (right click -> Delete Branch).
  • In the confirmation dialog, click "Delete Branch" button to proceed.
  • The deleted local branch item is still shown in the Branches view.

If you try to delete the branch again, you will get error message: "delete_local returned exit status 1".

So basically the branch had been deleted, but the git-cola UI still shows the branch in the list. The UI doesn't get updated after the delete branch is performed. You have to restart the git-cola to make the deleted branch disappear.


Also, could you add keyboard shortcut for deleting branch, maybe Shift + Delete ? and add support for multiple selection of the branch items, so I can delete them in one action.

@davvid davvid closed this as completed in 2066b98 Aug 1, 2019
@davvid
Copy link
Member

davvid commented Aug 1, 2019

Reopening for the Shift+Delete hotkey and multi-select request.

@ibraheem5022
Copy link

I'm on Git Cola v3.4 in manjaro linux.

How to produce the issue:

  • In the Branches view, delete one of the local branch item (right click -> Delete Branch).
  • In the confirmation dialog, click "Delete Branch" button to proceed.
  • The deleted local branch item is still shown in the Branches view.

If you try to delete the branch again, you will get error message: "delete_local returned exit status 1".

So basically the branch had been deleted, but the git-cola UI still shows the branch in the list. The UI doesn't get updated after the delete branch is performed. You have to restart the git-cola to make the deleted branch disappear.

Also, could you add keyboard shortcut for deleting branch, maybe Shift + Delete ? and add support for multiple selection of the branch items, so I can delete them in one action.

@sonulohani
Copy link

Hi @davvid ,

Could you please assign this issue to me. I want to contribute to this project and working on this issue would be a good start for me.

Thank you.

@sonulohani
Copy link

sonulohani commented May 17, 2020

This problem is due to the current branch, which is set to the one which you're trying to delete. Since the branch is current, so git can't delete the branch until the current branch is switched to some other branch. You can’t delete the branch you’re currently on.

@sonulohani
Copy link

Hi @davvid ,

As per my understanding no changes are required to make this fix as the application is working as expected. I am attaching the screenshot where you can see that git command error and the git-cola error is same and is what expected. I've already explained above on what causing the error. If you don't have any comments then we can close this issue.

git_error_screenshot

git_cola_error_screenshot

@s101d1
Copy link
Author

s101d1 commented May 18, 2020

Hi all,

I just tried it again with v3.6 and I can't reproduce the delete branch issue, so it seems the issue had been fixed.

@sonulohani It seems your issue is different than mine, in my case it's about deleting another branch (not the current one). But anyway, I can't reproduce your issue, on the right click menu for current branch, there is no "Delete Branch" menu at all. Perhaps you're using older version of git-cola? If you are able to reproduce the issue, you might want to create a new issuge page for that with detailed information on how to reproduce the issue.

@davvid I think this issue page can be closed. For Shift+Delete hotkey and multi-select request feature, you can just create a new separate issue page for that.

@davvid
Copy link
Member

davvid commented May 19, 2020

Thanks for verifying. I'm about to tag v3.7, thank you!

@davvid davvid closed this as completed May 19, 2020
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

4 participants