From b62ae16429dddefa5f765a747f57933cd7ce73c2 Mon Sep 17 00:00:00 2001 From: Dominic Gannaway Date: Wed, 19 Jun 2019 21:12:14 +0100 Subject: [PATCH] [Flare] Rename createEventComponent -> createEvent (#15929) --- packages/react-dom/src/client/ReactDOM.js | 4 ++-- .../__tests__/DOMEventResponderSystem-test.internal.js | 10 ++++------ packages/react-dom/src/fire/ReactFire.js | 6 ++++++ packages/react-events/README.md | 4 ++-- packages/react-events/src/Drag.js | 2 +- packages/react-events/src/Focus.js | 2 +- packages/react-events/src/FocusScope.js | 5 +---- packages/react-events/src/Hover.js | 2 +- packages/react-events/src/Press.js | 2 +- packages/react-events/src/Scroll.js | 5 +---- packages/react-events/src/Swipe.js | 2 +- 11 files changed, 21 insertions(+), 23 deletions(-) diff --git a/packages/react-dom/src/client/ReactDOM.js b/packages/react-dom/src/client/ReactDOM.js index 3bda0cd482..85816bb300 100644 --- a/packages/react-dom/src/client/ReactDOM.js +++ b/packages/react-dom/src/client/ReactDOM.js @@ -42,7 +42,7 @@ import { } from 'react-reconciler/inline.dom'; import {createPortal as createPortalImpl} from 'shared/ReactPortal'; import {canUseDOM} from 'shared/ExecutionEnvironment'; -import createEventComponent from 'shared/createEventComponent'; +import createEvent from 'shared/createEventComponent'; import {setBatchingImplementation} from 'events/ReactGenericBatching'; import { setRestoreImplementation, @@ -882,7 +882,7 @@ if (enableStableConcurrentModeAPIs) { } if (enableEventAPI) { - ReactDOM.unstable_createEventComponent = createEventComponent; + ReactDOM.unstable_createEvent = createEvent; } const foundDevTools = injectIntoDevTools({ diff --git a/packages/react-dom/src/events/__tests__/DOMEventResponderSystem-test.internal.js b/packages/react-dom/src/events/__tests__/DOMEventResponderSystem-test.internal.js index 1903f02b56..3344c729af 100644 --- a/packages/react-dom/src/events/__tests__/DOMEventResponderSystem-test.internal.js +++ b/packages/react-dom/src/events/__tests__/DOMEventResponderSystem-test.internal.js @@ -42,12 +42,10 @@ function createReactEventComponent({ allowMultipleHostChildren: allowMultipleHostChildren || false, }; - return { - $$typeof: Symbol.for('react.event_component'), - displayName: 'TestEventComponent', - props: null, - responder: testEventResponder, - }; + return ReactDOM.unstable_createEvent( + testEventResponder, + 'TestEventComponent', + ); } function dispatchEvent(element, type) { diff --git a/packages/react-dom/src/fire/ReactFire.js b/packages/react-dom/src/fire/ReactFire.js index bb74931fdc..d0af74edb2 100644 --- a/packages/react-dom/src/fire/ReactFire.js +++ b/packages/react-dom/src/fire/ReactFire.js @@ -70,6 +70,7 @@ import invariant from 'shared/invariant'; import lowPriorityWarning from 'shared/lowPriorityWarning'; import warningWithoutStack from 'shared/warningWithoutStack'; import {enableStableConcurrentModeAPIs} from 'shared/ReactFeatureFlags'; +import createEvent from 'shared/createEventComponent'; import { getInstanceFromNode, @@ -86,6 +87,7 @@ import { DOCUMENT_FRAGMENT_NODE, } from '../shared/HTMLNodeType'; import {ROOT_ATTRIBUTE_NAME} from '../shared/DOMProperty'; +import {enableEventAPI} from 'shared/ReactFeatureFlags'; const ReactCurrentOwner = ReactSharedInternals.ReactCurrentOwner; @@ -883,6 +885,10 @@ if (enableStableConcurrentModeAPIs) { ReactDOM.createSyncRoot = createSyncRoot; } +if (enableEventAPI) { + ReactDOM.unstable_createEvent = createEvent; +} + const foundDevTools = injectIntoDevTools({ findFiberByHostInstance: getClosestInstanceFromNode, bundleType: __DEV__ ? 1 : 0, diff --git a/packages/react-events/README.md b/packages/react-events/README.md index a56ba0a264..0e02720951 100644 --- a/packages/react-events/README.md +++ b/packages/react-events/README.md @@ -12,10 +12,10 @@ can be found [here](./docs). ## EventComponent -An Event Component is defined using `React.unstable_createEventComponent`: +An Event Component is defined using `ReactDOM.unstable_createEvent`: ```js -const EventComponent = React.unstable_createEventComponent( +const EventComponent = ReactDOM.unstable_createEvent( responder: EventResponder, displayName: string ); diff --git a/packages/react-events/src/Drag.js b/packages/react-events/src/Drag.js index 68e8be181d..032e8efc17 100644 --- a/packages/react-events/src/Drag.js +++ b/packages/react-events/src/Drag.js @@ -259,4 +259,4 @@ const DragResponder = { }, }; -export default ReactDOM.unstable_createEventComponent(DragResponder, 'Drag'); +export default ReactDOM.unstable_createEvent(DragResponder, 'Drag'); diff --git a/packages/react-events/src/Focus.js b/packages/react-events/src/Focus.js index a0a013e121..3992ded6a0 100644 --- a/packages/react-events/src/Focus.js +++ b/packages/react-events/src/Focus.js @@ -336,4 +336,4 @@ const FocusResponder = { }, }; -export default ReactDOM.unstable_createEventComponent(FocusResponder, 'Focus'); +export default ReactDOM.unstable_createEvent(FocusResponder, 'Focus'); diff --git a/packages/react-events/src/FocusScope.js b/packages/react-events/src/FocusScope.js index 668371c110..dd9209f2af 100644 --- a/packages/react-events/src/FocusScope.js +++ b/packages/react-events/src/FocusScope.js @@ -157,7 +157,4 @@ const FocusScopeResponder = { }, }; -export default ReactDOM.unstable_createEventComponent( - FocusScopeResponder, - 'FocusScope', -); +export default ReactDOM.unstable_createEvent(FocusScopeResponder, 'FocusScope'); diff --git a/packages/react-events/src/Hover.js b/packages/react-events/src/Hover.js index 4900617557..3efab72d13 100644 --- a/packages/react-events/src/Hover.js +++ b/packages/react-events/src/Hover.js @@ -406,4 +406,4 @@ const HoverResponder = { }, }; -export default ReactDOM.unstable_createEventComponent(HoverResponder, 'Hover'); +export default ReactDOM.unstable_createEvent(HoverResponder, 'Hover'); diff --git a/packages/react-events/src/Press.js b/packages/react-events/src/Press.js index 2a6f788aef..69007c03fd 100644 --- a/packages/react-events/src/Press.js +++ b/packages/react-events/src/Press.js @@ -970,4 +970,4 @@ const PressResponder = { }, }; -export default ReactDOM.unstable_createEventComponent(PressResponder, 'Press'); +export default ReactDOM.unstable_createEvent(PressResponder, 'Press'); diff --git a/packages/react-events/src/Scroll.js b/packages/react-events/src/Scroll.js index 9d5d53795e..27114ba51b 100644 --- a/packages/react-events/src/Scroll.js +++ b/packages/react-events/src/Scroll.js @@ -211,7 +211,4 @@ const ScrollResponder = { }, }; -export default ReactDOM.unstable_createEventComponent( - ScrollResponder, - 'Scroll', -); +export default ReactDOM.unstable_createEvent(ScrollResponder, 'Scroll'); diff --git a/packages/react-events/src/Swipe.js b/packages/react-events/src/Swipe.js index 44917e2186..9b6e74370d 100644 --- a/packages/react-events/src/Swipe.js +++ b/packages/react-events/src/Swipe.js @@ -262,4 +262,4 @@ const SwipeResponder = { }, }; -export default ReactDOM.unstable_createEventComponent(SwipeResponder, 'Swipe'); +export default ReactDOM.unstable_createEvent(SwipeResponder, 'Swipe');