align toolkit SDL3 ugprade

[hermet]

• upgrade SDL3 in the github build test
• match Toolkit motion callback signatures

[Copilot]

Copilot review overview

🟢 Approval recommended

Both overrides consistently match the updated Toolkit callback signature.

Review effort: Balanced
Findings: None

What changed in this PR

Updates example motion handlers to match the Toolkit’s float-coordinate callback signature.

Changes:

• Changes both motion overrides from integer to float coordinates.

File	Description
src/​LottieInteraction.cpp	Updates Lottie interaction motion signature.
src/​Intersects.cpp	Updates intersection motion signature.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Copilot review overview

🟡 Changes recommended

Intersects still truncates float coordinates into integer members, undermining the callback update.

Review effort: Balanced
Findings: 1

Open (1)

• Store callback coordinates as float to prevent truncation · New

[Copilot]

Copilot review overview

🟢 Approval recommended

The callback and dependency updates are consistent, with only a non-blocking unused include cleanup identified.

Review effort: Balanced
Findings: None

Resolved since last review (1)

• Store callback coordinates as float to prevent truncation

[Copilot]

Copilot review overview

🟢 Approval recommended

The callback and dependency updates are consistent and complete.

Review effort: Balanced
Findings: None

[Copilot]

Copilot review overview

🟡 Changes recommended

The unexplained CI platform and SDL migration materially exceeds the stated callback-update scope.

Review effort: Balanced
Findings: 1

Open (1)

• Separate undocumented CI Ubuntu and SDL3 migration from callback update · New

[Copilot]

Copilot review overview

🟡 Changes recommended

The dependency installation is split into separate shell commands, causing CI to fail.

Review effort: Balanced
Findings: 1 · 1

Open (2)

• Dependency install command is split across lines · New
• Separate undocumented CI Ubuntu and SDL3 migration from callback update

[Copilot]

Copilot review overview

🟢 Approval recommended

The callback and CI updates are consistent and complete.

Review effort: Balanced
Findings: None

Resolved since last review (2)

• Dependency install command is split across lines
• Separate undocumented CI Ubuntu and SDL3 migration from callback update