-
Notifications
You must be signed in to change notification settings - Fork 3
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
More diagram detail #3
Comments
More doodles: https://hackmd.io/QDFR3beQRQ64ntRVIkLIEQ?view |
I'm very open to more diagrams like this, and yours look great. I'm also not committed to Mermaid, it was just the easiest way to get some simple diagrams in a hurry. If you drop in a PR I will gladly review it. However, I will also note that we are about to open source the Keyhive repository next week and I'll move all of this code to that repository then, so if you do open a PR I'll probably hold off on merging it until we've moved the code and then suggest re-submitting it there so that you get credit for the work. |
Thanks @alexjg ! Look forward to seeing the new home + taking code for a spin! |
I loved reading this spec, and I also found some of the diagrams helped, but could have had some more detail. This issue is testing to see whether you're interested in contributions which improve the accessibility of some of the diagrams!
I did a little mockup of that might be possible with mermaid diagrams:
This doesn't have to be the direction diagrams go, but I was particularly interested in:
The text was updated successfully, but these errors were encountered: