Skip to content

ReactViewGroup post-conversion cleanup#46668

Closed
tdn120 wants to merge 3 commits into
facebook:mainfrom
tdn120:export-D63343964
Closed

ReactViewGroup post-conversion cleanup#46668
tdn120 wants to merge 3 commits into
facebook:mainfrom
tdn120:export-D63343964

Conversation

@tdn120
Copy link
Copy Markdown

@tdn120 tdn120 commented Sep 26, 2024

Summary:
Small tweaks post-Kotlin-conversion:

  • Make overflow a var
  • Replace + - with -
  • Clean up properties and move up init block
  • Iterate over entire allChildren array to clean up listeners

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D63343964

…pingViewManager to Kotlin""

Summary:
Original commit changeset: 9cbd8ab86be3

Original Phabricator Diff: D63076764

Differential Revision: D63129925
@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 Sep 26, 2024
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

Thomas Nardone added 2 commits September 26, 2024 10:33
Summary:
Original commit changeset: 01d3949c1312

Original Phabricator Diff: D63076763

Differential Revision: D63129923
Summary:
Pull Request resolved: facebook#46668

Small tweaks post-Kotlin-conversion:
- Make `overflow` a var
- Replace `+ -` with `-`
- Clean up properties and move up init block
- Iterate over entire allChildren array to clean up listeners

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D63343964
@facebook-github-bot
Copy link
Copy Markdown
Contributor

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

@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Sep 26, 2024
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request has been merged in 9cd83fa.

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 Merged This PR has been merged. p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants