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

Add support of toggling commit and author (name and date) #810

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

PaulChanHK
Copy link
Contributor

Add support of toggling commit and author (name and date)

i.e. toggling of:

  1. author and committer (toggle option=m)
  2. author-date and commit-date (toggle option=M)

Currently, applicable to view: main, log, refs, blame and tree.

@neeels
Copy link

neeels commented Jun 5, 2018

Tried this PR, rebased onto current upstream master, and noticed:

The 'm' shortcut is set for toggling the commit-date. It doesn't work, because m is already bound to 'main view'. 'M' doesn't work either, it says "Unknown key". Or I misunderstood that part.

When I'm in a help screen and issue the toggle then, and then exit the help screen, the dates have not toggled, and even though e.g. commit-date = no I'm still seeing the commit date. When I issue the toggle in the main view though the dates immediately change properly.

In the :h help screen the commit-date and committer options are not described.

It would be helpful if the commit log message(s) would say what config options have been added, so one doesn't need to parse the code to find out how to use the feature.

Commit 9bbe is just a comment tweak, but the commit log says something completely different.

@PaulChanHK
Copy link
Contributor Author

Add support of toggling commit and author (name and date)

1. author name and committer name, toggle option=m
2. author-date and commit-date, toggle option=M
This is effective in views: main, log, refs, blame and tree.

image

@PaulChanHK
Copy link
Contributor Author

PaulChanHK commented Oct 31, 2020

Tried this PR, rebased onto current upstream master, and noticed:

The 'm' shortcut is set for toggling the commit-date. It doesn't work, because m is already bound to 'main view'. 'M' doesn't work either, it says "Unknown key". Or I misunderstood that part.

When I'm in a help screen and issue the toggle then, and then exit the help screen, the dates have not toggled, and even though e.g. commit-date = no I'm still seeing the commit date. When I issue the toggle in the main view though the dates immediately change properly.

In the :h help screen the commit-date and committer options are not described.

It would be helpful if the commit log message(s) would say what config options have been added, so one doesn't need to parse the code to find out how to use the feature.

Commit 9bbe is just a comment tweak, but the commit log says something completely different.

  1. It have been working - You can access the "toggle options menu" by key 'o' and use L/R arrow keys to switch between the toggle options.
  2. New PR added an error when the Committer/Author are not effective in some views like stage view and diff view.
  3. The author did not add the "options menu" to the help, submitted another PR.

1. author name and committer name, toggle option=m
2. author-date and commit-date, toggle option=M
This is effective in views: main, log, refs, blame and tree.
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