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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: change keyboard toolbar offset instantly when keyboard is open #375

Conversation

IvanIhnatsiuk
Copy link
Contributor

@IvanIhnatsiuk IvanIhnatsiuk commented Mar 5, 2024

馃摐 Description

I've added logic to handle the case where the keyboard is open and the user focuses on another text input with a different input type.

The toolbar used to change animatedly but it didn't look good, now it changes just like the native toolbar.

馃挕 Motivation and Context

Closes #374

馃摙 Changelog

Changed logic in KeyboardStickyView component to handle case when keyboard is opened and keyboard size changed.

馃 How Has This Been Tested?

iPhone 15 pro

Pixel 3a emulator

Android 9 emulator

馃摳 Screenshots (if appropriate):

Pixel 3a emulator

Screen.Recording.2024-03-05.at.19.28.41.mov

Android 9

Screen.Recording.2024-03-05.at.19.41.00.mov

iPhone 15 pro

Simulator.Screen.Recording.-.iPhone.15.Pro.-.2024-03-05.at.19.28.29.mp4

馃摑 Checklist

  • CI successfully passed
  • I added new mocks and corresponding unit-tests if library API was changed

@kirillzyusko
Copy link
Owner

@IvanIhnatsiuk do you think it would be better to merge this PR instead? #376

@kirillzyusko kirillzyusko self-assigned this Mar 8, 2024
@kirillzyusko kirillzyusko self-requested a review March 8, 2024 07:54
@kirillzyusko
Copy link
Owner

Closed in favor of #376

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.

KeyboardToolbar component with different text input types
2 participants