Skip to content

Improvements to Git integration #6914#7053

Draft
nadment wants to merge 3 commits intoapache:mainfrom
nadment:6914
Draft

Improvements to Git integration #6914#7053
nadment wants to merge 3 commits intoapache:mainfrom
nadment:6914

Conversation

@nadment
Copy link
Copy Markdown
Contributor

@nadment nadment commented Apr 27, 2026

This PR isn't ready for a merge yet; I'm just looking for feedback.

@mattcasters
Copy link
Copy Markdown
Contributor

I really like the 2 new perspectives: commit and git.
Current functionality seems to work.
I like the graph view in the git perspective. I would love to see the "Visual Difference" option as a toolbar button when selecting a commit and a file in the graph tree.

@nadment
Copy link
Copy Markdown
Contributor Author

nadment commented Apr 29, 2026

Visual Diff appears as a button in the toolbar
image

@nadment nadment force-pushed the 6914 branch 3 times, most recently from 55b0721 to 0094fd3 Compare April 30, 2026 17:24
- Add Git commit perspective
- Add Git perspective
- TextDiffDialog
- Change shortcut of File Explorer perspective
Copy link
Copy Markdown
Contributor

@mattcasters mattcasters left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I built and tested the new perspectives and I absoutely love this work. Congrats.

@mattcasters
Copy link
Copy Markdown
Contributor

It should be easy to disable the 2 new perspectives in Hop Web until we find an alternative. Perhaps we can create a ticket for that separately.

@hansva
Copy link
Copy Markdown
Contributor

hansva commented May 4, 2026

Yes, if we want to ship this with 2.18 then that will be the best approach, I'd like to start the code freeze and a 2 week hardening/test period as we have done a massive amount of work.

@nadment
Copy link
Copy Markdown
Contributor Author

nadment commented May 4, 2026

I need to restore the commit function in GitGuiPlugin which is available in the file explorer for Hop web.

@hansva
Copy link
Copy Markdown
Contributor

hansva commented May 4, 2026

Right it's not as simple as just disabling the new views

@hansva
Copy link
Copy Markdown
Contributor

hansva commented May 4, 2026

But is the entire view jface or just specific elements that we could disable for hop web?

@nadment
Copy link
Copy Markdown
Contributor Author

nadment commented May 4, 2026

If we don't load the two perspectives in web mode (a hard-coded filter in HopGui until the next version) and I restore the menu/toolitem for the commit, it might work

@mattcasters
Copy link
Copy Markdown
Contributor

mattcasters commented May 4, 2026

If we don't load the two perspectives in web mode (a hard-coded filter in HopGui until the next version) and I restore the menu/toolitem for the commit, it might work

Exactly what I was thinking. Perhaps we can add an option in the annotation driving the perspective plugins for now.

@HopPerspectivePlugin something like boolean ignoreOnHopWeb() default false;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request]: Improvements to Git integration

3 participants