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

Prune remote branches from Remote repositories dialog not working as expected #6284

Closed
mast-eu opened this issue Feb 20, 2019 · 12 comments
Closed
Assignees
Milestone

Comments

@mast-eu
Copy link
Member

mast-eu commented Feb 20, 2019

Current behaviour

Clicking Prune remote branches in Repository > Remote repositories > Default pull behaviour (fetch & merge) prunes only one repository at a time. The pruned remote is the one selected in the first tab Remote repositories.

Expected behaviour

The button should trigger pruning of all remotes at once. Just like the checkbox Prune remote branches in the Pull dialog does.

Steps to reproduce

  1. Open Repository > Remote repositories. Note that I have 2 remotes configured.
    prune1
  2. Go to second tab Default pull behaviour (fetch & merge) and click Prune remote branches.
    prune2
  3. Verify in the output that only one remote is actually pruned.
    prune4
  4. Go back to first tab Remote repositories and select the other remote.
  5. Go back to second tab and click Prune remote branches again. Now the other remote gets pruned.

Did this work in previous version of GitExtensions

No. I think it was always like this.

Environment

  • Git Extensions 3.0.2.5232
  • Build 23b6f51
  • Git 2.20.1.windows.1
  • Microsoft Windows NT 10.0.17763.0
  • .NET Framework 4.7.3324.0
  • DPI 96dpi (no scaling)
@vbjay
Copy link
Contributor

vbjay commented Feb 20, 2019 via email

@mast-eu
Copy link
Member Author

mast-eu commented Feb 20, 2019

That doesn't seem logical to me, because you have no way of figuring out which remote it refers to (unless you switch tabs). Moreover, the button next to it Update all remote branch info works on all remotes.

@vbjay
Copy link
Contributor

vbjay commented Feb 20, 2019

Frankly, I use the pull dialog
image

@vbjay
Copy link
Contributor

vbjay commented Feb 20, 2019

Or
image

@gerhardol
Copy link
Member

I do not see this as @mast-eu but can see that this is misunderstood too.
One way to improve this is to move the button closer to "Save changes" (where the remote is seen) or the first tab (already crowded though).

@mast-eu
Copy link
Member Author

mast-eu commented Feb 20, 2019

One way to improve this is to move the button closer to "Save changes" (where the remote is seen)

Sorry, but the "Remote repository" seen above the Save changes button is only for changing the second column in the table on the left side, isn't it? It is not connected to Prune remote branches.

@vbjay
Copy link
Contributor

vbjay commented Feb 20, 2019

Correct.

@pmiossec
Copy link
Member

One way to improve this is to move the button closer to "Save changes" (where the remote is seen) or the first tab

For me, the best way to improve the situation is to remove completely the button. I never found it a good place to "hide" this feature in a form that is more a "configuration" form where few people go. It takes myself years to discover that GitExtensions proposed this feature and used to did it by command lines...

And now that we have the left panel, we could add this feature in the contextual menu of the remotes and in the one of the "Remotes" node to propose a "prune all remotes" feature.

@RussKie
Copy link
Member

RussKie commented Feb 21, 2019

@pmiossec sounds good!

@RussKie RussKie added area: user experience up-for-grabs Easy tasks for those looking to get involved. Refer to https://up-for-grabs.net/ 🤓 good first issue labels Feb 21, 2019
@pmiossec
Copy link
Member

I already did a quick try. I have to do the PR...

@pmiossec
Copy link
Member

PR done: #6293
@mast-eu you could test...

@mast-eu
Copy link
Member Author

mast-eu commented Feb 22, 2019

@mast-eu you could test...

Yes, tonight I'll have a look at it.

@ghost ghost assigned RussKie Feb 25, 2019
@ghost ghost removed the status: ready label Feb 25, 2019
@RussKie RussKie removed 🤓 good first issue type: discussion up-for-grabs Easy tasks for those looking to get involved. Refer to https://up-for-grabs.net/ labels Feb 25, 2019
@RussKie RussKie added this to the 3.1.0 milestone Feb 25, 2019
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

5 participants