fix(sankey): keep explicitly-positioned and snapped nodes inside the plot area - #7978

Closed

CAOShurong wants to merge 15 commits into

plotly:mainfrom

CAOShurong:codex/sankey-7946-clip-nodes-in-bounds

Commits on Aug 25, 2026

fix(sankey): derive effective node.pad clamp from layout geometry
CAOShurong
committed
3f29ecb
fix(sankey): declare loop counter locally; add draftlog
CAOShurong
committed
a95a89c
test(sankey): drive pad-warning tests by explicit layout size
CAOShurong
committed
3ab85f8
test(sankey): use star topology so clamping is actually exercised
CAOShurong
committed
1752e5d
test(sankey): collect all Lib.warn arguments in the spy
CAOShurong
committed
900ff34
test(sankey): size the non-clamped case to genuinely fit node.pad
CAOShurong
committed
a79b770
test(sankey): raise the non-clamped case height to 1000
CAOShurong
committed
e11a889
test(sankey): drop stray layoutversion key from the mock
CAOShurong
committed
d70753c
test(sankey): use a short sink column for the non-clamped case
CAOShurong
committed
741761c
fix(sankey): declare loop counter locally
CAOShurong
committed
386d241
fix(sankey): keep explicitly-positioned and snapped nodes inside the plot area (plotly.js #7946)
CAOShurong
committed
a477b49
docs(draftlog): add changelog entry for #7978
CAOShurong
committed
6675782
ci: empty push to retrigger flaky jasmine shards
CAOShurong
committed
13c4240
ci: retrigger flaky no-gl-jasmine shard
CAOShurong
committed
455da46

Commits on Aug 29, 2026

fix(sankey): drop duplicate node.pad patch from clip fix
CAOShurong
committed
db3e9f1