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

Improve graph rendering in the default graphic mode #985

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

Conversation

proski
Copy link
Contributor

@proski proski commented Jan 19, 2020

*  Improve graph rendering in the default graphic mode
   
   Using the same symbol for ordinary and boundary commits is misleading.
   Keep 'o' for boundary commits only, it's similar to other graphic modes.
   
   Use ACS_BULLET for ordinary commits, ACS_DIAMOND for merges and '@' for
   the initial commit, which is similar to the symbols used in the UTF-8
   graphic mode.
   
   Update tests for the new graph symbols. ACS_BULLET corresponds to '~' in
   the alternative charset.

@proski proski force-pushed the better-default-graph branch from cca1c0a to b696dc1 Compare January 19, 2020 04:39
Using the same symbol for ordinary and boundary commits is misleading.
Keep 'o' for boundary commits only, it's similar to other graphic modes.

Use ACS_BULLET for ordinary commits, ACS_DIAMOND for merges and '@' for
the initial commit, which is similar to the symbols used in the UTF-8
graphic mode.

Update tests for the new graph symbols. ACS_BULLET corresponds to '~' in
the alternative charset.
@proski proski force-pushed the better-default-graph branch from b696dc1 to 037d7f3 Compare January 19, 2020 04:43
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.

1 participant