Skip to content

Commit

Permalink
fix: Update incorrect SwitchChangeEvent type
Browse files Browse the repository at this point in the history
  • Loading branch information
retyui committed Oct 11, 2022
1 parent e4dff28 commit 609e1cf
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ import * as React from 'react';

type SwitchChangeEvent = $ReadOnly<{|
value: boolean,
target: number,
|}>;

type NativeProps = $ReadOnly<{|
Expand Down
6 changes: 5 additions & 1 deletion Libraries/Components/Switch/Switch.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ import {NativeMethods} from '../../Renderer/shims/ReactNativeTypes';
import {ColorValue, StyleProp} from '../../StyleSheet/StyleSheet';
import {ViewStyle} from '../../StyleSheet/StyleSheetTypes';
import {ViewProps} from '../View/ViewPropTypes';
import {NativeSyntheticEvent, TargetedEvent} from '../../Types/CoreEventTypes';

export interface SwitchPropsIOS extends ViewProps {
/**
Expand All @@ -37,10 +38,13 @@ export interface SwitchPropsIOS extends ViewProps {
tintColor?: ColorValue | undefined;
}

export interface SwitchChangeEvent extends React.SyntheticEvent {
export interface SwitchChangeEventData extends TargetedEvent {
value: boolean;
}

export interface SwitchChangeEvent
extends NativeSyntheticEvent<SwitchChangeEventData> {}

export interface SwitchProps extends SwitchPropsIOS {
/**
* Color of the foreground switch grip.
Expand Down
1 change: 1 addition & 0 deletions Libraries/Components/Switch/Switch.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ import * as React from 'react';
type SwitchChangeEvent = SyntheticEvent<
$ReadOnly<{|
value: boolean,
target: number,
|}>,
>;

Expand Down
1 change: 1 addition & 0 deletions Libraries/Components/Switch/SwitchNativeComponent.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ import * as React from 'react';

type SwitchChangeEvent = $ReadOnly<{|
value: boolean,
target: number,
|}>;

type NativeProps = $ReadOnly<{|
Expand Down
2 changes: 1 addition & 1 deletion types/__typetests__/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -1429,7 +1429,7 @@ const SwitchOnChangePromiseTest = () => (
<Switch
onChange={event => {
const e: SwitchChangeEvent = event;
return new Promise(() => e.value);
return new Promise(() => e.nativeEvent.value);
}}
/>
);
Expand Down

0 comments on commit 609e1cf

Please sign in to comment.