Skip to content

Commit

Permalink
fix waffle stroke (when constant) (#2204)
Browse files Browse the repository at this point in the history
Co-authored-by: Mike Bostock <[email protected]>
  • Loading branch information
Fil and mbostock authored Nov 21, 2024
1 parent eb85419 commit 5fb8616
Show file tree
Hide file tree
Showing 9 changed files with 757 additions and 51 deletions.
2 changes: 1 addition & 1 deletion src/marks/waffle.js
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,7 @@ function waffleRender({render, ...options}) {
.attr("transform", template`translate(${x0},${y0})`)
.attr("d", (i) => `M${polygon[i].join("L")}Z`)
.attr("fill", (i) => `url(#${patternId}-${i})`)
.attr("stroke", this.stroke == null ? null : (i) => `url(#${patternId}-${i})`)
.attr("stroke", this.stroke == null ? null : "none")
.call(applyChannelStyles, this, {ariaLabel, href, title})
)
.node();
Expand Down
40 changes: 20 additions & 20 deletions test/output/waffleShapes.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
24 changes: 12 additions & 12 deletions test/output/waffleStroke.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
12 changes: 6 additions & 6 deletions test/output/waffleStrokeMixed.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading

0 comments on commit 5fb8616

Please sign in to comment.