Skip to content

Minimum pinch#155

Closed
elliottkember wants to merge 1 commit intomasterfrom
elliott/minimum-pinch
Closed

Minimum pinch#155
elliottkember wants to merge 1 commit intomasterfrom
elliott/minimum-pinch

Conversation

@elliottkember
Copy link
Copy Markdown
Collaborator

@elliottkember elliottkember commented Dec 25, 2025

Set a minimum of 44pt to avoid crazy math happening at really close pinch sizes


Note

Modernizes the CommonJS build and component implementation without altering core behavior.

  • Migrates rendering to the react/jsx-runtime across ReactNativeZoomableView, AnimatedTouchFeedback, StaticPin, and debug helpers
  • Converts many methods to class fields/arrow functions; moves defaultProps to static class property and updates exports
  • Replaces callback guards with optional chaining, refactors PanResponder handlers, and tidies event invocation/debouncing
  • Minor structure/typing cleanups in helpers and animations; no functional changes intended

Written by Cursor Bugbot for commit e91e589. Configure here.

@elliottkember elliottkember mentioned this pull request Dec 25, 2025
@thomasttvo thomasttvo changed the base branch from master to thomas/reanimated December 29, 2025 18:35
@thomasttvo thomasttvo added the bug Something isn't working label Jan 2, 2026
@elliottkember
Copy link
Copy Markdown
Collaborator Author

elliottkember commented Feb 25, 2026

@thomasttvo I think moving this branch to the reanimated branch broke something here. Not sure. It shouldn't be a billion lines

@elliottkember elliottkember changed the base branch from thomas/reanimated to master February 25, 2026 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants