Skip to content

feat: add PolyStar shape support - #102

Merged
RobertBrewitz merged 1 commit into
linebender:mainfrom
RobertBrewitz:feat-polystar
Apr 22, 2026
Merged

feat: add PolyStar shape support#102
RobertBrewitz merged 1 commit into
linebender:mainfrom
RobertBrewitz:feat-polystar

Conversation

@RobertBrewitz

@RobertBrewitz RobertBrewitz commented Apr 21, 2026

Copy link
Copy Markdown
Member

Wires PolyStarShape into AnyShape under "sr" and adds a Geometry::Star variant backed by animated::Star, matching how Rect and Ellipse are handled.

AI disclosure

I only implemented support for fixed PolyStarShape last year without animation support. While splitting out my source to multiple chunks of changes, I had Claude Opus 4.7 generate PolyStarTest.json and write the the evaluation code for Star so we also get animation support.

@RobertBrewitz
RobertBrewitz force-pushed the feat-polystar branch 2 times, most recently from faf079b to 06970d4 Compare April 21, 2026 08:32
@RobertBrewitz
RobertBrewitz requested a review from nuzzles April 21, 2026 08:55
Wires `PolyStarShape` into `AnyShape` under `"sr"` and adds a
`Geometry::Star` variant backed by `animated::Star`, matching how `Rect`
and `Ellipse` are handled.
@RobertBrewitz
RobertBrewitz added this pull request to the merge queue Apr 22, 2026
Merged via the queue into linebender:main with commit 2cc4210 Apr 22, 2026
16 checks passed
@RobertBrewitz
RobertBrewitz deleted the feat-polystar branch April 22, 2026 07:24
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.

2 participants