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

Boxes + horizontal flow makes for more readable call graphs #2603

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

Conversation

DanielVF
Copy link

@DanielVF DanielVF commented Nov 12, 2024

This is my personal set of modifications to the call graph generation. Putting them here to see if anyone else likes them.

Boxes vs ellipses

By having nodes be boxes instead of ellipses, the information packs down so much denser. The "circle" style wastes huge amounts of space on either side. This change is an easy win.

Horizontal flow vs vertical.

This one is more opinionated. When there are more than five or so elements in a given layer, horizontal flow allows much more to be onscreen at once, since the nodes stack short side to short side. Since almost everything I do is looking at full contracts with this tool, this works out much better for me. Vertical ranking means that even on a widescreen monitor, I can't see the whole contract.

Using vertical ranking. (Old, but with boxes)

image

Using horizontal. (After)

image

@CLAassistant
Copy link

CLAassistant commented Nov 12, 2024

CLA assistant check
All committers have signed the CLA.

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.

2 participants