Skip to content

Commit 08f624b

Browse files
committed
Refine event registration + event signatures
1 parent 26071ab commit 08f624b

File tree

5 files changed

+10
-14
lines changed

5 files changed

+10
-14
lines changed

packages/react-dom/src/events/DOMModernPluginEventSystem.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -632,7 +632,7 @@ function createDispatchQueueItem(
632632
};
633633
}
634634

635-
export function accumulateTwoPhaseListeners(
635+
export function accumulatePhaseListeners(
636636
targetFiber: Fiber | null,
637637
dispatchQueue: DispatchQueue,
638638
event: ReactSyntheticEvent,

packages/react-dom/src/events/plugins/ModernBeforeInputEventPlugin.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ import {
2929
} from '../FallbackCompositionState';
3030
import SyntheticCompositionEvent from '../SyntheticCompositionEvent';
3131
import SyntheticInputEvent from '../SyntheticInputEvent';
32-
import {accumulateTwoPhaseListeners} from '../DOMModernPluginEventSystem';
32+
import {accumulatePhaseListeners} from '../DOMModernPluginEventSystem';
3333

3434
const END_KEYCODES = [9, 13, 27, 32]; // Tab, Return, Esc, Space
3535
const START_KEYCODE = 229;
@@ -241,7 +241,7 @@ function extractCompositionEvent(
241241
nativeEvent,
242242
nativeEventTarget,
243243
);
244-
accumulateTwoPhaseListeners(targetInst, dispatchQueue, event);
244+
accumulatePhaseListeners(targetInst, dispatchQueue, event);
245245

246246
if (fallbackData) {
247247
// Inject data generated from fallback path into the synthetic event.
@@ -411,7 +411,7 @@ function extractBeforeInputEvent(
411411
nativeEvent,
412412
nativeEventTarget,
413413
);
414-
accumulateTwoPhaseListeners(targetInst, dispatchQueue, event);
414+
accumulatePhaseListeners(targetInst, dispatchQueue, event);
415415
event.data = chars;
416416
}
417417

packages/react-dom/src/events/plugins/ModernChangeEventPlugin.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ import {disableInputAttributeSyncing} from 'shared/ReactFeatureFlags';
3737
import {batchedUpdates} from '../ReactDOMUpdateBatching';
3838
import {
3939
dispatchEventsInBatch,
40-
accumulateTwoPhaseListeners,
40+
accumulatePhaseListeners,
4141
} from '../DOMModernPluginEventSystem';
4242

4343
function registerEvents() {
@@ -63,7 +63,7 @@ function createAndAccumulateChangeEvent(
6363
event.type = 'change';
6464
// Flag this event loop as needing state restore.
6565
enqueueStateRestore(((target: any): Node));
66-
accumulateTwoPhaseListeners(inst, dispatchQueue, event);
66+
accumulatePhaseListeners(inst, dispatchQueue, event);
6767
}
6868
/**
6969
* For IE shims

packages/react-dom/src/events/plugins/ModernSelectEventPlugin.js

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ import {
2929
} from '../../client/ReactDOMComponentTree';
3030
import {hasSelectionCapabilities} from '../../client/ReactInputSelection';
3131
import {DOCUMENT_NODE} from '../../shared/HTMLNodeType';
32-
import {accumulateTwoPhaseListeners} from '../DOMModernPluginEventSystem';
32+
import {accumulatePhaseListeners} from '../DOMModernPluginEventSystem';
3333

3434
const skipSelectionChangeEvent =
3535
canUseDOM && 'documentMode' in document && document.documentMode <= 11;
@@ -136,11 +136,7 @@ function constructSelectEvent(dispatchQueue, nativeEvent, nativeEventTarget) {
136136
syntheticEvent.type = 'select';
137137
syntheticEvent.target = activeElement;
138138

139-
accumulateTwoPhaseListeners(
140-
activeElementInst,
141-
dispatchQueue,
142-
syntheticEvent,
143-
);
139+
accumulatePhaseListeners(activeElementInst, dispatchQueue, syntheticEvent);
144140
}
145141
}
146142

packages/react-dom/src/events/plugins/ModernSimpleEventPlugin.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ import {
2323
registerSimpleEvents,
2424
} from '../DOMEventProperties';
2525
import {
26-
accumulateTwoPhaseListeners,
26+
accumulatePhaseListeners,
2727
accumulateEventHandleTargetListeners,
2828
} from '../DOMModernPluginEventSystem';
2929
import {IS_TARGET_PHASE_ONLY} from '../EventSystemFlags';
@@ -166,7 +166,7 @@ function extractEvents(
166166
inCapturePhase,
167167
);
168168
} else {
169-
accumulateTwoPhaseListeners(targetInst, dispatchQueue, event, true);
169+
accumulatePhaseListeners(targetInst, dispatchQueue, event, true);
170170
}
171171
return event;
172172
}

0 commit comments

Comments
 (0)