Skip to content

Commit

Permalink
Revert "Fix ref type for Native Scroll View"
Browse files Browse the repository at this point in the history
This reverts commit db662af.
  • Loading branch information
grabbou committed Oct 28, 2019
1 parent 32c90eb commit 2fac662
Showing 1 changed file with 22 additions and 7 deletions.
29 changes: 22 additions & 7 deletions Libraries/Components/ScrollView/ScrollView.js
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ import type {
ScrollEvent,
LayoutEvent,
} from '../../Types/CoreEventTypes';
import type {HostComponent} from '../../Renderer/shims/ReactNativeTypes';
import type {State as ScrollResponderState} from '../ScrollResponder';
import type {ViewProps} from '../View/ViewPropTypes';
import type {Props as ScrollViewStickyHeaderProps} from './ScrollViewStickyHeader';
Expand Down Expand Up @@ -770,11 +769,15 @@ class ScrollView extends React.Component<Props, State> {
return ReactNative.findNodeHandle(this._innerViewRef);
}

getInnerViewRef(): ?React.ElementRef<HostComponent<mixed>> {
getInnerViewRef(): ?React.ElementRef<
Class<ReactNative.NativeComponent<mixed>>,
> {
return this._innerViewRef;
}

getNativeScrollRef(): ?React.ElementRef<HostComponent<mixed>> {
getNativeScrollRef(): ?React.ElementRef<
Class<ReactNative.NativeComponent<mixed>>,
> {
return this._scrollViewRef;
}
Expand Down Expand Up @@ -947,13 +950,21 @@ class ScrollView extends React.Component<Props, State> {
this.props.onContentSizeChange(width, height);
};

_scrollViewRef: ?React.ElementRef<HostComponent<mixed>> = null;
_setScrollViewRef = (ref: ?React.ElementRef<HostComponent<mixed>>) => {
_scrollViewRef: ?React.ElementRef<
Class<ReactNative.NativeComponent<mixed>>,
> = null;
_setScrollViewRef = (
ref: ?React.ElementRef<Class<ReactNative.NativeComponent<mixed>>>,
) => {
this._scrollViewRef = ref;
};

_innerViewRef: ?React.ElementRef<HostComponent<mixed>> = null;
_setInnerViewRef = (ref: ?React.ElementRef<HostComponent<mixed>>) => {
_innerViewRef: ?React.ElementRef<
Class<ReactNative.NativeComponent<mixed>>,
> = null;
_setInnerViewRef = (
ref: ?React.ElementRef<Class<ReactNative.NativeComponent<mixed>>>,
) => {
this._innerViewRef = ref;
};

Expand Down Expand Up @@ -1057,6 +1068,7 @@ class ScrollView extends React.Component<Props, State> {
const contentContainer = (
<ScrollContentContainerViewClass
{...contentSizeChangeProps}
// $FlowFixMe Invalid prop usage
ref={this._setInnerViewRef}
style={contentContainerStyle}
removeClippedSubviews={
Expand Down Expand Up @@ -1166,6 +1178,7 @@ class ScrollView extends React.Component<Props, State> {
// On iOS the RefreshControl is a child of the ScrollView.
// tvOS lacks native support for RefreshControl, so don't include it in that case
return (
// $FlowFixMe
<ScrollViewClass {...props} ref={this._setScrollViewRef}>
{Platform.isTV ? null : refreshControl}
{contentContainer}
Expand All @@ -1184,13 +1197,15 @@ class ScrollView extends React.Component<Props, State> {
<ScrollViewClass
{...props}
style={[baseStyle, inner]}
// $FlowFixMe
ref={this._setScrollViewRef}>
{contentContainer}
</ScrollViewClass>,
);
}
}
return (
// $FlowFixMe
<ScrollViewClass {...props} ref={this._setScrollViewRef}>
{contentContainer}
</ScrollViewClass>
Expand Down

0 comments on commit 2fac662

Please sign in to comment.