Skip to content

Diagram enhancements #578

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Diagram enhancements #578

wants to merge 7 commits into from

Conversation

pau-hedgehog
Copy link
Contributor

fixes some cosmetic bugs (overlapping labels) and improves diagram rendering for more complex topologies, such as 3 spine ones

@pau-hedgehog pau-hedgehog self-assigned this May 5, 2025
@pau-hedgehog pau-hedgehog requested review from a team as code owners May 5, 2025 09:45
@pau-hedgehog pau-hedgehog force-pushed the pau/fix_diagram_labels branch 2 times, most recently from aba7bd4 to c4f796b Compare May 5, 2025 21:08
@pau-hedgehog pau-hedgehog force-pushed the pau/fix_diagram_labels branch from c4f796b to a7628e0 Compare May 6, 2025 13:34
@pau-hedgehog pau-hedgehog requested a review from Frostman May 6, 2025 13:41
@pau-hedgehog pau-hedgehog force-pushed the pau/fix_diagram_labels branch 2 times, most recently from 9db6eec to ed23ae2 Compare May 6, 2025 16:29
@pau-hedgehog pau-hedgehog force-pushed the pau/fix_diagram_labels branch 3 times, most recently from 5a5e832 to 0fc5e00 Compare May 13, 2025 11:45
@pau-hedgehog pau-hedgehog requested a review from a team as a code owner May 13, 2025 11:45
@pau-hedgehog pau-hedgehog force-pushed the pau/fix_diagram_labels branch 2 times, most recently from 8f7d79b to 72a648d Compare May 14, 2025 13:20
changes labels to text boxes instead of inline

fixes vertical edge label overlap and better style

Signed-off-by: Pau Capdevila <pau@githedgehog.com>
- Improve edge routing with better connection point distribution
- Enhance label positioning for better readability
- Fix overlapping connections with offsets
- Generate legend based on connections present

Signed-off-by: Pau Capdevila <pau@githedgehog.com>
groups parallel spine connections for clarity

Signed-off-by: Pau Capdevila <pau@githedgehog.com>
add legend to mermaid diagram

Signed-off-by: Pau Capdevila <pau@githedgehog.com>
adds output name option

adds just targets for different vlab diagrams
adds just target for lab-ci env

Signed-off-by: Pau Capdevila <pau@githedgehog.com>
Signed-off-by: Pau Capdevila <pau@githedgehog.com>
@pau-hedgehog pau-hedgehog force-pushed the pau/fix_diagram_labels branch from 72a648d to 8540b5c Compare May 16, 2025 10:55
Fixes #640

Signed-off-by: Pau Capdevila <pau@githedgehog.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants