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

Align logic in BaseTextInputShadowNode to determine updateStateIfNeeded with AndroidTextInputShadowNode #48585

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

christophpurrer
Copy link
Contributor

Summary:
[Changelog] [Internal] - Align logic in BaseTextInputShadowNode to determine updateStateIfNeeded with AndroidTextInputShadowNode

As a preparation for #48165 this aligns the implementation of those 2 methods

Differential Revision: D68004755

…putShadowNode for Android (facebook#48582)

Summary:

[Changelog] [Internal] - Preparation for sharing common ShadowNode functionality in BaseTextInputShadowNode for Android

As a preparation for facebook#48165 this change aligns the order of methods between:
- BaseTextInputShadowNode.h
- AndroidTextInputShadowNode.h

to make it easier for future changes to look at the delta between both implementations.

The goal is land facebook#48582 which aligns the RN iOS and RN Android implementation

Differential Revision: D68001423
…g with AndroidTextInputShadowNode (facebook#48584)

Summary:

[Changelog] [Internal] - Align logic in BaseTextInputShadowNode to calculate placeholder string with AndroidTextInputShadowNode

As a preparation for facebook#48165 this aligns the implementation of those 2 methods

Differential Revision: D68004218
…ed with AndroidTextInputShadowNode

Summary:
[Changelog] [Internal] - Align logic in BaseTextInputShadowNode to determine updateStateIfNeeded with AndroidTextInputShadowNode

As a preparation for facebook#48165 this aligns the implementation of those 2 methods

Differential Revision: D68004755
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Jan 10, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D68004755

christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Jan 10, 2025
…tateIfNeeded with AndroidTextInputShadowNode (facebook#48585)

Summary:

[Changelog] [Internal] - Align logic in BaseTextInputShadowNode to determine updateStateIfNeeded with AndroidTextInputShadowNode

As a preparation for facebook#48165 this aligns the implementation of those 2 methods

Differential Revision: D68004755
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request Jan 10, 2025
…ed with AndroidTextInputShadowNode (facebook#48585)

Summary:

[Changelog] [Internal] - Align logic in BaseTextInputShadowNode to determine updateStateIfNeeded with AndroidTextInputShadowNode

As a preparation for facebook#48165 this aligns the implementation of those 2 methods

Differential Revision: D68004755
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. fb-exported p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants