Skip to content

Commit a63b770

Browse files
j-piaseckimeta-codesync[bot]
authored andcommitted
Upgrade Modal visibility to "for frameworks" (#58285)
Summary: Pull Request resolved: #58285 Reclassifies `react/renderer/components/modal:modal` from public to "for frameworks" under the C++ stable API three-tier visibility model. Consumers that opt into `RN_STRICT_API` now get a suppressible warning where they previously got an error pointing at the umbrella, and can acknowledge it with `RN_ALLOW_FRAMEWORKS`; without that flag the guards are inert, so no existing build changes behaviour. Changelog: [Internal] Reviewed By: christophpurrer Differential Revision: D118440328
1 parent 1c66137 commit a63b770

7 files changed

Lines changed: 4 additions & 52 deletions

File tree

packages/react-native/ReactCommon/React-FabricComponents.podspec

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -95,12 +95,6 @@ Pod::Spec.new do |s|
9595
sss.header_dir = "react/renderer/components/modal"
9696
end
9797

98-
ss.subspec "modalUmbrella" do |sss|
99-
sss.source_files = "react/renderer/components/modal/React/*.h"
100-
sss.header_dir = ""
101-
sss.header_mappings_dir = "react/renderer/components/modal"
102-
end
103-
10498
ss.subspec "safeareaview" do |sss|
10599
sss.source_files = podspec_sources("react/renderer/components/safeareaview/**/*.{m,mm,cpp,h}", "react/renderer/components/safeareaview/**/*.h")
106100
# Exclude tests to avoid conflicts with the react-native-safe-area-context package

packages/react-native/ReactCommon/react/renderer/components/modal/ModalHostViewComponentDescriptor.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
#pragma once
99

10-
#include <react/cxxstableapi/UmbrellaGuard.h>
10+
#include <react/cxxstableapi/FrameworksGuard.h>
1111

1212
#include <glog/logging.h>
1313
#include <react/renderer/components/modal/ModalHostViewShadowNode.h>

packages/react-native/ReactCommon/react/renderer/components/modal/ModalHostViewShadowNode.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
#pragma once
99

10-
#include <react/cxxstableapi/UmbrellaGuard.h>
10+
#include <react/cxxstableapi/FrameworksGuard.h>
1111

1212
#include <react/renderer/components/FBReactNativeSpec/EventEmitters.h>
1313
#include <react/renderer/components/FBReactNativeSpec/Props.h>

packages/react-native/ReactCommon/react/renderer/components/modal/ModalHostViewState.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
#pragma once
99

10-
#include <react/cxxstableapi/UmbrellaGuard.h>
10+
#include <react/cxxstableapi/FrameworksGuard.h>
1111

1212
#include <react/renderer/core/graphicsConversions.h>
1313
#include <react/renderer/graphics/Float.h>

packages/react-native/ReactCommon/react/renderer/components/modal/ModalHostViewUtils.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
#pragma once
99

10-
#include <react/cxxstableapi/UmbrellaGuard.h>
10+
#include <react/cxxstableapi/FrameworksGuard.h>
1111

1212
#include <react/renderer/graphics/Size.h>
1313

packages/react-native/ReactCommon/react/renderer/components/modal/React/Modal.h

Lines changed: 0 additions & 36 deletions
This file was deleted.

packages/react-native/scripts/ios-prebuild/headers-config.js

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -357,12 +357,6 @@ const PodspecExceptions /*: {[key: string]: PodSpecConfiguration} */ = {
357357
headerDir: 'react/renderer/components/modal',
358358
},
359359

360-
{
361-
name: 'modalUmbrella',
362-
headerPatterns: ['react/renderer/components/modal/React/*.h'],
363-
headerDir: 'React',
364-
},
365-
366360
{
367361
name: 'safeareaview',
368362
headerPatterns: ['react/renderer/components/safeareaview/**/*.h'],

0 commit comments

Comments
 (0)