Skip to content

fix(etch): edges hidden behind containers#53

Merged
avrabe merged 1 commit intomainfrom
fix/edge-visibility
Mar 20, 2026
Merged

fix(etch): edges hidden behind containers#53
avrabe merged 1 commit intomainfrom
fix/edge-visibility

Conversation

@avrabe
Copy link
Copy Markdown
Contributor

@avrabe avrabe commented Mar 20, 2026

Edges rendered before containers, so container backgrounds painted over them. Fix: containers → edges → leaf nodes.

Render containers → edges → leaf nodes (was: edges → all nodes).
Add port label margin to prevent clipping.

Trace: skip

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@avrabe avrabe merged commit 28a9896 into main Mar 20, 2026
12 checks passed
@avrabe avrabe deleted the fix/edge-visibility branch March 20, 2026 05:18
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'Rivet Criterion Benchmarks'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.

Benchmark suite Current: 9e05dab Previous: c1886da Ratio
link_graph_build/10000 38240638 ns/iter (± 4294183) 28349124 ns/iter (± 2377978) 1.35
validate/10000 6324820 ns/iter (± 872950) 5107676 ns/iter (± 473444) 1.24
diff/10000 9668341 ns/iter (± 778058) 7765996 ns/iter (± 422529) 1.24

This comment was automatically generated by workflow using github-action-benchmark.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@avrabe avrabe restored the fix/edge-visibility branch March 20, 2026 05:48
@avrabe avrabe deleted the fix/edge-visibility branch March 20, 2026 06:04
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.

1 participant