-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Add option to open submodule from diff lists #2706
Comments
+1 Yes, my team runs into this "issue" as well. You double click it hoping It would be even better if the superproject's diff view would also show
|
I agree that showing the diff information in the same window would be helpful. But, at the same time I think it makes sense to provide the ability to open the submodule from that list even if that information is displayed. Avoiding the need to hunt through the source tree as you described for more information that isn't imperative to a diff summary. |
Implemented by #2740 |
New implementation is pretty nice. I'd like to add a detail to improve it further: Newly opened GitExtensions window jumps to the related commit in the submodule. How about if it jumped to (selected them) the 2 commits (from commit and to commit) so that we can directly see the total changes in that submodule by the commit in the super-repo? |
I like that idea. Please, raise it as a new feature request. |
Done! Thanks. |
Some of the repositories I work with have a lot of submodules that are fairly active. Navigating those changes is fairly frustrating because currently all that the diff shows is the commit reference change and what files changed in that commit. There is also no direct way to open that submodule from the list of changes to get more information. It would be great if double clicking a submodule in a diff list would open a new browse window for that submodule with the referenced commit selected so that all the information can quickly be found (commit, tree, and diff). Adding this as an option seems adequate.
I am willing to do the work on this one.
The text was updated successfully, but these errors were encountered: