Skip to content

[iOS] Fabric: Fixes insets not adjust when keyboard disappear#47924

Closed
zhongwuzw wants to merge 1 commit into
react:mainfrom
zhongwuzw:features/fix_fabric_scrollview_inset
Closed

[iOS] Fabric: Fixes insets not adjust when keyboard disappear#47924
zhongwuzw wants to merge 1 commit into
react:mainfrom
zhongwuzw:features/fix_fabric_scrollview_inset

Conversation

@zhongwuzw

Copy link
Copy Markdown
Contributor

Summary:

Fixes #47731 .

Changelog:

[IOS] [FIXED] - Fabric: Fixes insets not adjust when keyboard disappear

Test Plan:

Demo in #47731

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 25, 2024
@facebook-github-bot facebook-github-bot added the Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. label Nov 25, 2024

if (CGRectIsNull(viewIntersection)) {
return;
}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Paper infra doesn't have this quick return logic.

@cipolleschi cipolleschi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thanks for looking into this.

@facebook-github-bot

Copy link
Copy Markdown
Contributor

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Dec 4, 2024
@facebook-github-bot

Copy link
Copy Markdown
Contributor

@cipolleschi merged this pull request in efcfe5d.

@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @zhongwuzw in efcfe5d

When will my fix make it into a release? | How to file a pick request?

blakef pushed a commit that referenced this pull request Dec 9, 2024
Summary:
Fixes #47731 .

[IOS] [FIXED] - Fabric: Fixes insets not adjust when keyboard disappear

Pull Request resolved: #47924

Test Plan: Demo in #47731

Reviewed By: blakef

Differential Revision: D66651865

Pulled By: cipolleschi

fbshipit-source-id: a75afbd1a7651f0c77022d913f910821c482fcf7
@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @zhongwuzw in d105c2c

When will my fix make it into a release? | How to file a pick request?

blakef pushed a commit that referenced this pull request Dec 9, 2024
Summary:
Fixes #47731 .

## Changelog:

[IOS] [FIXED] - Fabric: Fixes insets not adjust when keyboard disappear

Pull Request resolved: #47924

Test Plan: Demo in #47731

Reviewed By: blakef

Differential Revision: D66651865

Pulled By: cipolleschi

fbshipit-source-id: a75afbd1a7651f0c77022d913f910821c482fcf7
@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @zhongwuzw in 19bff65

When will my fix make it into a release? | How to file a pick request?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

automaticallyAdjustKeyboardInsets Does Not adjust Insets When Keyboard Disappears again

4 participants