Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(iOS): implement scaleX and scaleY transition animations. #7807

Merged
merged 2 commits into from
Apr 8, 2024

Conversation

oblador
Copy link
Contributor

@oblador oblador commented Nov 2, 2023

I was tearing my hair over why I couldn't get scale transitions to work on iOS while they were working fine on Android, and it turns out it was never implemented 😅

This PR adds this implementation.

Fixes #7066

@oblador oblador changed the title iOS scaleX/scaleY transition feat(transition): iOS scaleX/scaleY transition Nov 2, 2023
@oblador
Copy link
Contributor Author

oblador commented Nov 7, 2023

Not sure about the failing tests, seems unrelated write error: No space left on device

Copy link

stale bot commented Mar 17, 2024

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the 🏚 stale label Mar 17, 2024
@oblador
Copy link
Contributor Author

oblador commented Mar 17, 2024

Bad bot

@stale stale bot removed the 🏚 stale label Mar 17, 2024
@oblador
Copy link
Contributor Author

oblador commented Apr 8, 2024

@asafkorem Any chance this could get some eyes on it?

@asafkorem asafkorem self-requested a review April 8, 2024 08:49
@asafkorem asafkorem self-assigned this Apr 8, 2024
@asafkorem
Copy link
Collaborator

Yes @oblador, I'll review it later this week

@asafkorem asafkorem changed the title feat(transition): iOS scaleX/scaleY transition fix(iOS): implement scaleX and scaleY transition animations. Apr 8, 2024
Copy link
Collaborator

@asafkorem asafkorem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for this contribution @oblador, great work! 💯

@oblador
Copy link
Contributor Author

oblador commented Apr 8, 2024

Awesome, thanks for the review! Would you mind delegating another PR that I’ve had open for quite some time? #7809

@asafkorem asafkorem merged commit 020a94a into wix:master Apr 8, 2024
7 of 8 checks passed
@asafkorem
Copy link
Collaborator

Sure @oblador

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.

setRoot animations don't work on iOS, was it designed that way?
2 participants