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: currentColor overflow #2570

Merged
merged 2 commits into from
Dec 10, 2024
Merged

fix: currentColor overflow #2570

merged 2 commits into from
Dec 10, 2024

Conversation

jakex7
Copy link
Member

@jakex7 jakex7 commented Dec 9, 2024

Summary

This is a blind fix for issue #2566 that resets the caller on prepareForRecycle. While it should help address the overflow happening in some cases, I cannot guarantee that it will fully resolve the issue due to the lack of a reproducible scenario.

Compatibility

OS Implemented
iOS
MacOS

@jakex7 jakex7 merged commit 856142e into main Dec 10, 2024
4 of 6 checks passed
@jakex7 jakex7 deleted the @jakex7/fixCurrentColorOverflow branch December 10, 2024 10:15
jakex7 added a commit that referenced this pull request Dec 16, 2024
# Summary
This is another fix for issue
#2566 that
also resets the color on prepareForRecycle (in the same manner of this
fix #2570). I
also don't have a sandboxed reproduce, but was able to verify the fix on
my company's code which I unfortunately can't share.

## Compatibility
OS | Implemented
-- | --
iOS | ✅
MacOS | ✅

Co-authored-by: Jakub Grzywacz <[email protected]>
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.

1 participant