Skip to content

fix: dashes + stroke apply to elbow arrows in the style panel - #115

Merged
Amark19 merged 1 commit into
masterfrom
fix/elbow-arrow-style-panel
Aug 24, 2026
Merged

fix: dashes + stroke apply to elbow arrows in the style panel#115
Amark19 merged 1 commit into
masterfrom
fix/elbow-arrow-style-panel

Conversation

@Amark19

@Amark19 Amark19 commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

What changed

applyToActive in the properties panel only styled the line connector, so an elbow arrow (whose connector is a polyline) was skipped — changing dash, width, or colour on a selected elbow arrow did nothing to the line (only the head recoloured). Now it includes polyline.

One line + comment. Off master, targets master — independent of the open elbow PRs (#113/#114).

Test plan

  • Verified in-browser: select an elbow arrow → apply dashed and thick → the polyline now gets strokeDashArray + strokeWidth and renders as a thick dashed elbow (before: no change). Straight arrows and shapes unaffected.
  • 85 unit tests pass; lint + prettier clean.

🤖 Generated with Claude Code

applyToActive only styled the 'line' connector, skipping an elbow arrow's
'polyline' connector — so changing dash/width/colour on a selected elbow arrow
did nothing to the line (only the head recoloured). Include polyline.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying whiteboard with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9dd1882
Status: ✅  Deploy successful!
Preview URL: https://d6d94faf.whiteboard-988.pages.dev
Branch Preview URL: https://fix-elbow-arrow-style-panel.whiteboard-988.pages.dev

View logs

@Amark19
Amark19 merged commit c6f9d69 into master Aug 24, 2026
2 checks passed
@Amark19
Amark19 deleted the fix/elbow-arrow-style-panel branch August 24, 2026 13:56
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