Skip to content

Move ReactNativeFeatureFlags to src/private#52610

Closed
coado wants to merge 1 commit into
react:mainfrom
coado:export-D75448698
Closed

Move ReactNativeFeatureFlags to src/private#52610
coado wants to merge 1 commit into
react:mainfrom
coado:export-D75448698

Conversation

@coado

@coado coado commented Jul 15, 2025

Copy link
Copy Markdown
Contributor

Summary:
This diff removes ReactNativeFeatureFlags from react-native/Libraries/ReactNative and migrates
shouldPressibilityUseW3CPointerEventsForHover to common ReactNativeFeatureFlags in src/private/featureflags. The shouldEmitW3CPointerEvents is removed as it is used in rn-tester to hide some examples.

Changelog:
[General][Breaking] - Migrate shouldPressibilityUseW3CPointerEventsForHover to common private feature flags and remove shouldEmitW3CPointerEvents flag.

Reviewed By: robhogan

Differential Revision: D75448698

@meta-cla meta-cla 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 Jul 15, 2025
@facebook-github-bot facebook-github-bot added p: Software Mansion Partner: Software Mansion Partner p: Facebook Partner: Facebook labels Jul 15, 2025
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@coado coado force-pushed the export-D75448698 branch from f17c142 to ceb1e1d Compare July 21, 2025 13:42
Summary:
Pull Request resolved: #52610

This diff removes `ReactNativeFeatureFlags` from `react-native/Libraries/ReactNative` and migrates
`shouldPressibilityUseW3CPointerEventsForHover` to common `ReactNativeFeatureFlags` in `src/private/featureflags`. The `shouldEmitW3CPointerEvents is removed as it is used in `rn-tester` to hide some examples.

Changelog:
[General][Breaking] - Migrate `shouldPressibilityUseW3CPointerEventsForHover` to common private feature flags and remove `shouldEmitW3CPointerEvents` flag.

Reviewed By: robhogan

Differential Revision: D75448698
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@coado coado force-pushed the export-D75448698 branch from ceb1e1d to eb808f9 Compare July 21, 2025 13:47
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Jul 23, 2025
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request has been merged in fb45877.

@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @coado in fb45877

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

CarSmallGuo pushed a commit to CarSmallGuo/ohos_react_native_new that referenced this pull request Mar 25, 2026
Co-authored-by: Patryk Fryda<patryk.fryda@swmansion.com>



# message auto-generated for no-merge-commit merge:
!1843 merge 0.82 into 0.82

chore: update react-native to 0.82.1

Created-by: pafry
Commit-by: Patryk Fryda
Merged-by: huangyouhua
Description: # https://gitcode.com/openharmony-sig/ohos_react_native/merge_requests/1835 has to be merged first.


## Why are these changes being made?
There is ongoing effort to update react-native and RNOH to 0.82. This is the last step.

### Notable react-native changes:
- react/react-native#51600 - infoLog removal
- react/react-native#53094 - changing libhermes to libhermesvm
- react/react-native#52610 - moving ReactNativeFeatureFlags to src/private
- react/react-native#53360 - stable DOM API
- react/react-native#52750 - Install ContextContainer::Shared with std::shared_ptr

### Other changes:
- replaced custom RNOH cxx turbomodules

## Changelog
- updated react-native to 0.82.1

## Test Plan
- launch tester app,
- run sequential and concurrent tests

## Checklist
- [x] `pnpm verify` command has passed successfully.
- [x] No failures occurred in automatic tests on the `CONCURRENT TESTER` page in the tester app.
- [x] No failures occurred in automatic tests on the `SEQUENTIAL TESTER` page in the tester app.
- [x] MR does not include a breaking change or a review has been requested from a Huawei reviewer.

See merge request: openharmony-sig/ohos_react_native!1843
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 p: Software Mansion Partner: Software Mansion Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants