Skip to content

Commit 13a2b11

Browse files
p-sunkelset
authored andcommitted
Minor: Rename AccessibilityInfo.sendAccessibilityEvent_unstable to sendAccessibilityEvent
Summary: Changelog: [Internal] Rename AccessibilityInfo.sendAccessibilityEvent_unstable to sendAccessibilityEvent In Fabric, we want people to use `AccessibilityInfo.sendAccessibilityEvent` instead of `UIManager.sendAccessibilityEvent` for Android. The API is not unstable. There is a test in [AccessibilityExample.js](https://github.com/facebook/react-native/blob/c940eb0c49518b82a3999dcac3027aa70018c763/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js#L959) in RNTester to confirm that it works. A search for [`AccessibilityInfo.sendAccessibilityEvent_unstable` in Github](https://github.com/search?q=AccessibilityInfo.sendAccessibilityEvent_unstable&type=Code) shows that it's not being used yet, which makes sense because it's an Fabric API. Therefore it's safe to rename it. Reviewed By: sammy-SC Differential Revision: D37901006 fbshipit-source-id: 73f35b09ca8f9337f4d66a431f0a3f815da38249
1 parent 2de4cf0 commit 13a2b11

File tree

4 files changed

+4
-4
lines changed

4 files changed

+4
-4
lines changed

Libraries/Components/AccessibilityInfo/AccessibilityInfo.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -345,7 +345,7 @@ const AccessibilityInfo: AccessibilityInfoType = {
345345
/**
346346
* Send a named accessibility event to a HostComponent.
347347
*/
348-
sendAccessibilityEvent_unstable(
348+
sendAccessibilityEvent(
349349
handle: ElementRef<HostComponent<mixed>>,
350350
eventType: AccessibilityEventTypes,
351351
) {

Libraries/ReactNative/BridgelessUIManager.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ module.exports = {
146146
errorMessageForMethod('setLayoutAnimationEnabledExperimental'),
147147
);
148148
},
149-
// Please use AccessibilityInfo.sendAccessibilityEvent_unstable instead.
149+
// Please use AccessibilityInfo.sendAccessibilityEvent instead.
150150
// See SetAccessibilityFocusExample in AccessibilityExample.js for a migration example.
151151
sendAccessibilityEvent: (reactTag: ?number, eventType: number): void =>
152152
console.error(errorMessageForMethod('sendAccessibilityEvent')),

jest/setup.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ jest
133133
isScreenReaderEnabled: jest.fn(() => Promise.resolve(false)),
134134
removeEventListener: jest.fn(),
135135
setAccessibilityFocus: jest.fn(),
136-
sendAccessibilityEvent_unstable: jest.fn(),
136+
sendAccessibilityEvent: jest.fn(),
137137
getRecommendedTimeoutMillis: jest.fn(),
138138
},
139139
}))

packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -956,7 +956,7 @@ function SetAccessibilityFocusExample(props: {}): React.Node {
956956

957957
const onPress = () => {
958958
if (myRef && myRef.current) {
959-
AccessibilityInfo.sendAccessibilityEvent_unstable(myRef.current, 'focus');
959+
AccessibilityInfo.sendAccessibilityEvent(myRef.current, 'focus');
960960
}
961961
};
962962

0 commit comments

Comments
 (0)