FazBrowse GitHub Viewer | Trending | | Home

Tools: [Download Repo ZIP] [Original HTTPS Page]

fix(sankey): derive effective node.pad clamp from layout geometry by CAOShurong · Pull Request #7977 · plotly/plotly.js · GitHub

fix(sankey): derive effective node.pad clamp from layout geometry - #7977

Open

CAOShurong wants to merge 11 commits into

plotly:mainfrom

CAOShurong:codex/sankey-7832-effective-padding

Commits on Aug 25, 2026

fix(sankey): derive effective node.pad clamp from layout geometry
CAOShurong
committed
5fb6575
fix(sankey): declare loop counter locally; add draftlog
CAOShurong
committed
46737ec
test(sankey): drive pad-warning tests by explicit layout size
CAOShurong
committed
7c6adaa
test(sankey): use star topology so clamping is actually exercised
CAOShurong
committed
c277ec7
test(sankey): collect all Lib.warn arguments in the spy
CAOShurong
committed
708b03d
test(sankey): size the non-clamped case to genuinely fit node.pad
CAOShurong
committed
17b036f
test(sankey): raise the non-clamped case height to 1000
CAOShurong
committed
c6821f7
test(sankey): drop stray layoutversion key from the mock
CAOShurong
committed
31eb47d
test(sankey): use a short sink column for the non-clamped case
CAOShurong
committed
43b5579

Commits on Aug 29, 2026

test(sankey): repair pad warning test placement
CAOShurong
committed
b0687ab
fix(sankey): tolerate padding comparison rounding
CAOShurong
committed
edd00fb

Back | FazBrowse Home | New Git URL