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

ShowFullHistory in View menu #9803

Merged

Conversation

gerhardol
Copy link
Member

Related to #9796

Proposed changes

It is possible to configure the following in FormFileHistory, but the settings also affect the normal git-log.
The effect on normal git-log is not great, but it is there.

https://git-scm.com/docs/git-log#Documentation/git-log.txt---full-history
https://git-scm.com/docs/git-log#Documentation/git-log.txt---simplify-merges

If View had a submenu, they could be put there (but one submenu with two items is not much gain).

This could also be in Settings, similar to above.

Something should be put in the doc as well, probably here (in the 4.0 PR draft).
There are quite some things that should be added to the doc though, I will not do all of that.

https://git-extensions-documentation.readthedocs.io/en/feature-4.0-prepare/browse_repository.html#main-toolbar
gitextensions/GitExtensionsDoc#140

Screenshots

Before

This is not changed
image

RevGrid context menu in FileHistory, removed as they are added to View.
image

After

FormBrowse
image

All RevGrid View menus
image

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.

@gerhardol gerhardol merged commit 2c910cc into gitextensions:master Jan 8, 2022
@ghost ghost added this to the vNext milestone Jan 8, 2022
@gerhardol gerhardol deleted the feature/i9796-view-full-history branch January 8, 2022 18:06
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.

None yet

2 participants