Skip to content

anchor support for scaleEffect and rotationEffect#302

Merged
marcprux merged 2 commits intoskiptools:mainfrom
vincentborko:anchor-support
Jan 17, 2026
Merged

anchor support for scaleEffect and rotationEffect#302
marcprux merged 2 commits intoskiptools:mainfrom
vincentborko:anchor-support

Conversation

@vincentborko
Copy link
Contributor

@vincentborko vincentborko commented Jan 16, 2026

Previous state: Worked but anchor was ignored
Change: Use graphicsLayer with TransformOrigin instead of Modifier.scale() or Modifier.rotate()

Edited: Fixed wording

  • REQUIRED: I have signed the Contributor Agreement
  • REQUIRED: I have tested my change locally with swift test
  • OPTIONAL: I have tested my change on an iOS simulator or device
  • OPTIONAL: I have tested my change on an Android emulator or device

Current state: Works but anchor is ignored
Change: Use graphicsLayer with TransformOrigin instead of Modifier.scale() or Modifier.rotate()
@cla-bot cla-bot bot added the cla-signed label Jan 16, 2026
@marcprux
Copy link
Member

This will be great!

One request: can you convert this PR to a "draft" until it is working and ready for review?

@vincentborko
Copy link
Contributor Author

Do you mean when the workflow tests pass?
everything is working for me, I added a playground for it as well
image

except for this empty test always

@vincentborko
Copy link
Contributor Author

Complement to skiptools/skipapp-showcase-fuse#36

@marcprux
Copy link
Member

Oh, I see. I interpreted "Works but anchor is ignored" as meaning that you were going to try getting it working :) No problem.

Can you update the scaleEffect doc in the README.md to indicate that it is working and add a note that the anchor is ignored?

skip-ui/README.md

Lines 1919 to 1931 in 23672af

<tr>
<td>🟡</td>
<td>
<details>
<summary><code>.scaleEffect</code></summary>
<ul>
<li><code>func scaleEffect(_ scale: CGSize) -> some View</code></li>
<li><code>func scaleEffect(_ s: CGFloat) -> some View</code></li>
<li><code>func scaleEffect(x: CGFloat = 1.0, y: CGFloat = 1.0) -> some View</code></li>
</ul>
</details>
</td>
</tr>

@vincentborko
Copy link
Contributor Author

Ahh, nvm I just worded it badly. "Current state" means pre-commit. Anchor works now, I'll update the readme

@marcprux
Copy link
Member

Another great contribution! Thanks!

@marcprux marcprux merged commit 8670c9c into skiptools:main Jan 17, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants