Skip to content

Commit

Permalink
Improve graph rendering in the default graphic mode
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
Pavel Roskin committed Jan 18, 2020
1 parent ca0809d commit cca1c0a
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions src/graph-v1.c
Original file line number Diff line number Diff line change
Expand Up @@ -396,11 +396,11 @@ graph_symbol_to_chtype(const struct graph_symbol *symbol)
if (symbol->boundary)
graphics[1] = 'o';
else if (symbol->initial)
graphics[1] = 'I';
graphics[1] = '@';
else if (symbol->merge)
graphics[1] = 'M';
graphics[1] = ACS_DIAMOND;
else
graphics[1] = 'o'; //ACS_DIAMOND; //'*';
graphics[1] = ACS_BULLET;
return graphics;
}

Expand Down
6 changes: 3 additions & 3 deletions src/graph-v2.c
Original file line number Diff line number Diff line change
Expand Up @@ -1096,11 +1096,11 @@ graph_symbol_to_chtype(const struct graph_symbol *symbol)
if (symbol->boundary)
graphics[1] = 'o';
else if (symbol->initial)
graphics[1] = 'I';
graphics[1] = '@';
else if (symbol->merge)
graphics[1] = 'M';
graphics[1] = ACS_DIAMOND;
else
graphics[1] = 'o'; //ACS_DIAMOND; //'*';
graphics[1] = ACS_BULLET;
return graphics;

} else if (graph_symbol_cross_merge(symbol)) {
Expand Down

0 comments on commit cca1c0a

Please sign in to comment.