Skip to content

fix: setting multiple transform attributes at once - #2816

Merged
jakex7 merged 4 commits into
software-mansion:mainfrom
kacperzolkiewski:fixMultipleTransforms
Nov 12, 2025
Merged

jakex7 merged 4 commits into
software-mansion:mainfrom
kacperzolkiewski:fixMultipleTransforms

Conversation

@kacperzolkiewski

Copy link
Copy Markdown
Collaborator

Summary

Fixes: #2686

  • Remove the transformsArrayToProps function, which did not handle transforms correctly.
  • Add a new stringifyTransformArrayProps function that converts a transform array into an SVG-compatible string (e.g. "rotate(45) translate(10)".

Result

Before:
image

After:
image

Test Plan

Run the example from issue #2686 and verify if Rect displays correctly.

Compatibility

OS Implemented
iOS ✅
MacOS ✅
Android ✅
Web ✅

Checklist

  • I have tested this on a device and a simulator
  • I added documentation in README.md
  • I updated the typed files (typescript)
  • I added a test for the API in the __tests__ folder

Comment thread src/lib/extract/extractTransform.ts Resolved
Comment thread src/lib/extract/extractTransform.ts Resolved
Comment thread src/lib/extract/extractTransform.ts Resolved
Comment thread src/lib/extract/extractTransform.ts Resolved
Comment thread src/lib/extract/extractTransform.ts Resolved
@jakex7
jakex7 merged commit ebe6366 into software-mansion:main Nov 12, 2025
3 checks passed
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.

Multiple attributes in the transform attribute will not take effect

2 participants