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

Select remote branch on Alt+Click in side panel #10083

Merged

Conversation

gerhardol
Copy link
Member

@gerhardol gerhardol commented Jul 24, 2022

Fixes #9194

Proposed changes

When Alt+Click on local branches in the sidepanel, select the tracked remote branch.
Simplify finding remote tracked branches (to for instance find differences).

For branches with no tracked remote the local is selected as before.

Note Ctrl+Alt+Click is not handled now. Should that toggle the remote branch status?
(commit message is not correct right now)

Discussed in #9194.
Cherry-picked from @mstv private branch, with a minor adjustment

Screenshots

Before

The local branch was always selected.

After

One normal click selecting the same branch, one alt-click to select the remote branch revision.

i9194-alt-click

Test methodology

Manual

Merge strategy

I agree that the maintainer squash merge this PR (if the commit message is clear).


✒️ I contribute this code under The Developer Certificate of Origin.

@ghost ghost assigned gerhardol Jul 24, 2022
@gerhardol gerhardol changed the title Select remote branch on [Ctrl+]Alt+Click in side panel Select remote branch on Alt+Click in side panel Jul 24, 2022
@RussKie
Copy link
Member

RussKie commented Jul 25, 2022

I'd like an easy way to toggle branch highlighting too - #8452

@gerhardol
Copy link
Member Author

I'd like an easy way to toggle branch highlighting too - #8452

That is not a conflict, if we agree on how to use the available combinations.
I agree with the following comment for the draft PR, Alt-Click should always activate.
#8452 (review)

Ctrl-Alt-Click (in both left panel and grid) need to be agreed on to (with current PRs it is like Ctrl-Click in side panel and Click in the grid).

@gerhardol
Copy link
Member Author

Ctrl-Alt-Click (in both left panel and grid) need to be agreed on to (with current PRs it is like Ctrl-Click in side panel and Click in the grid).

#8452 is no longer a draft, Alt-LClick is always Highlighting. This PR also handles Alt independent of Ctrl, no change needed.

@gerhardol gerhardol requested a review from mstv July 27, 2022 20:31
@RussKie RussKie added this to the 4.0.0 milestone Jul 28, 2022
@gerhardol gerhardol merged commit cabe0d2 into gitextensions:master Jul 28, 2022
@gerhardol gerhardol deleted the feature/rot-left-click-related branch July 28, 2022 20:01
@ghost ghost modified the milestones: 4.0.0, vNext Jul 28, 2022
gerhardol added a commit that referenced this pull request Jul 28, 2022
Co-authored-by: Gerhard Olsson <gerhardol@users.noreply.github.com>
(cherry picked from commit cabe0d2)
@gerhardol gerhardol modified the milestones: vNext, 4.0.0 Sep 4, 2022
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

Successfully merging this pull request may close these issues.

Left Panel: Jump between tracked branches
3 participants