Skip to content

Stagger pin labels in staircase layout#9

Merged
xbst merged 2 commits into
xbst:masterfrom
Ntrondle:staircase-pin-labels
Jul 15, 2026
Merged

Stagger pin labels in staircase layout#9
xbst merged 2 commits into
xbst:masterfrom
Ntrondle:staircase-pin-labels

Conversation

@Ntrondle

@Ntrondle Ntrondle commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Summary
place top/bottom pin labels on separate stepped levels
extend their leader lines to match each label level
keep left/right pin labels vertically aligned because they are already separated
expand the SVG padding so staggered labels are not clipped
keep the Python generator and browser designer renderers in sync

Why
Dense connectors previously placed labels on one shared baseline and reduced the font size to fit the available pin pitch. This made adjacent labels difficult to distinguish and increased the risk of associating a definition with the wrong pin.

The staircase layout preserves a readable label size and gives horizontally arranged pins visually distinct leaders. Connectors whose labels already form a vertical list keep equal leader lengths.

Validation
python -m compileall -q pinout_gen/pinout_gen
node --check pinout_design/js/svg-renderer.js
generated pinout_gen/example.toml successfully
checked that top/bottom orientations stair-step and left/right orientations remain aligned
checked a two-row connector
visually inspected the generated HTML

@Ntrondle Ntrondle marked this pull request as ready for review July 15, 2026 06:06
Copilot AI review requested due to automatic review settings July 15, 2026 06:06

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Ntrondle Ntrondle marked this pull request as draft July 15, 2026 06:08
@Ntrondle Ntrondle marked this pull request as ready for review July 15, 2026 06:08
@xbst xbst merged commit d992a0f into xbst:master Jul 15, 2026
@xbst

xbst commented Jul 15, 2026

Copy link
Copy Markdown
Owner

Thanks for implementing this, helps a lot with long pin names on horizontal connectors. Merged.

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