Skip to content

Wip 81 log graph - #2628

Draft
peso wants to merge 34 commits into
gitui-org:masterfrom
peso:wip-81-log-graph
Draft

Wip 81 log graph#2628
peso wants to merge 34 commits into
gitui-org:masterfrom
peso:wip-81-log-graph

Conversation

@peso

@peso peso commented Apr 29, 2025

Copy link
Copy Markdown
Contributor

This Pull Request is wip towards fixing #81

It changes the following:

  • work-in-progrss for adding git-graph DO NOT MERGE

Lots of bugs and strange behaviour

peso added 30 commits April 29, 2025 05:36
This should be merged with the commit that replaces dummy code with first attempt
@peso
peso marked this pull request as draft May 3, 2025 07:17
@kooshi

kooshi commented May 23, 2025

Copy link
Copy Markdown

Hey man, if you accomplish this, you will be my hero. Thanks for tackling it!

@peso

peso commented Jun 23, 2025

Copy link
Copy Markdown
Contributor Author

Right now I'm working on refactoring git-graph so the git backend can be swapped. It is based on git2 and gitui is moving away from git2 towards gix.

@yk-kd

yk-kd commented Jul 14, 2025

Copy link
Copy Markdown

Fonts can be made configurable to display graphs more neatly.
So it would be great if you could make the letter of the join configurable.

kovidgoyal/kitty#7681
https://github.com/rbong/vim-flog
https://www.reddit.com/r/neovim/comments/1ec118n/git_graph_teaser/

@peso

peso commented Jul 14, 2025

Copy link
Copy Markdown
Contributor Author

The symbols are already configurable. git-graph (not the vs code plugin "git graph") has a configurable option called "style" to change the symbols used.

Regarding the specific symbols you link to, git-graph issue 90 requests the enhancement you propose. Note however, that the proposed symbol-set is larger than the current symbol-set and thus require some changes at the algorithm level. It is a bit more work than simply changing the symbols.

@yk-kd

yk-kd commented Jul 14, 2025

Copy link
Copy Markdown

Sorry if I'm wrong.

-s, --style <style>      Output style. One of [normal/thin|round|bold|double|ascii].

I thought style could only be set up from these five, since it was written above.

@peso

peso commented Jul 15, 2025

Copy link
Copy Markdown
Contributor Author

You are right. As a user you can only change symbols by picking one of the predefined styles. There is no support for changing a single symbol (eg. "join") in a style.

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.

3 participants