Skip to content

Update design for port graphs - #1007

Open
AleksBelytskyi wants to merge 1 commit into
inex:mainfrom
AleksBelytskyi:feature/update_port_graph_desing
Open

Update design for port graphs#1007
AleksBelytskyi wants to merge 1 commit into
inex:mainfrom
AleksBelytskyi:feature/update_port_graph_desing

Conversation

@AleksBelytskyi

Copy link
Copy Markdown
Contributor

With this PR, I would like to propose a new design for port graphs in IXP Manager, as the current one is based on the old-fashioned MRTG style.

I welcome any feedback on this new layout. Please let me know your thoughts on the design choices or if there are any specific tweaks you'd like to see before merging.

@afk11

afk11 commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Hi @AleksBelytskyi, thanks for opening a pull request!

I have some sample data to test this out on, so sharing some screenshots for review

Without changes:
Screenshot 2026-07-28 at 09 24 58

With changes:
Screenshot 2026-07-28 at 09 18 20

@nickhilliard

Copy link
Copy Markdown
Member

It would be useful if the png described which colour scheme applied to each direction. In the original version of the p2p graphing code, instead of "In" or "Out", we used "$srcname -> $dstname".

I.e. NREN -> AS112 instead of In and AS112 -> NREN instead of Out.

This needed careful truncation because some organisation names are very long.

@barryo

barryo commented Jul 28, 2026

Copy link
Copy Markdown
Member

I kinda like the current/old way better 😬

@nickhilliard

Copy link
Copy Markdown
Member

hrm, I have no idea what "In" and "Out" mean. Every time I look at a graph, I end up scratching my head, wondering which way is which.

@barryo

barryo commented Jul 28, 2026

Copy link
Copy Markdown
Member

hrm, I have no idea what "In" and "Out" mean.

I don't mean I think in/out betterm just the general design. For sure change in/out to something more obvious.

Also Abbreviated Name is customer exists for shortened names for graphs, etc.

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.

4 participants