Skip to content

fix: stabilize keyboard layout tracking - #1539

Closed
MarshallBear1 wants to merge 3 commits into
kirillzyusko:mainfrom
MarshallBear1:codex/fix-keyboard-layout-regressions
Closed

fix: stabilize keyboard layout tracking#1539
MarshallBear1 wants to merge 3 commits into
kirillzyusko:mainfrom
MarshallBear1:codex/fix-keyboard-layout-regressions

Conversation

@MarshallBear1

@MarshallBear1 MarshallBear1 commented Jul 9, 2026

Copy link
Copy Markdown

Summary

  • prevent KeyboardAvoidingView padding modes from feeding their animated layout back into the keyboard-height calculation
  • tolerate one physical pixel of rounding when iOS matches interactive keyboard positions with fractional accessory offsets
  • reset animated, Reanimated, and focused-input state when KeyboardController is disabled mid-transition
  • invoke the example app bundle scripts with separately quoted paths, so builds work from checkout paths containing spaces

Validation

  • yarn jest src/__tests__/animated.spec.tsx src/components/KeyboardAvoidingView/__tests__/initialFrame.spec.tsx --runInBand
  • yarn typescript
  • yarn lint
  • built the iOS example app for iOS 26.4
  • launched it on iPhone 17 Pro Max and exercised the automatic KeyboardAvoidingView keyboard open/dismiss flow

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

📊 Package size report

Current size Target Size Difference
334233 bytes 332793 bytes 1440 bytes 📈

@kirillzyusko

Copy link
Copy Markdown
Owner

Hey @MarshallBear1

Could you split your PR into 4 smaller PRs?

For me it looks like you are fixing 4 different problems in a single PR, I'll not be able to merge it. Correct fix for AI comments already landed here: #1542

@MarshallBear1

Copy link
Copy Markdown
Author

Split as requested:

The duplicate AI-review workflow change was dropped because #1542 already landed. Thanks for the direction.

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.

2 participants