Skip to content

Commit 7ba7acd

Browse files
elicwhitefacebook-github-bot
authored andcommitted
Surfacing Flow issues around invalid props
Reviewed By: yungsters Differential Revision: D7977386 fbshipit-source-id: a6df2d75e0caa55a84f9c9c6860f622942955dfc
1 parent 65c336f commit 7ba7acd

File tree

5 files changed

+8
-9
lines changed

5 files changed

+8
-9
lines changed

Libraries/Components/ScrollView/ScrollView.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -237,6 +237,7 @@ const ScrollView = createReactClass({
237237
* - `false`, deprecated, use 'never' instead
238238
* - `true`, deprecated, use 'always' instead
239239
*/
240+
// $FlowFixMe
240241
keyboardShouldPersistTaps: PropTypes.oneOf([
241242
'always',
242243
'never',
@@ -521,9 +522,11 @@ const ScrollView = createReactClass({
521522

522523
UNSAFE_componentWillMount: function() {
523524
this._scrollAnimatedValue = new AnimatedImplementation.Value(
525+
// $FlowFixMe
524526
this.props.contentOffset ? this.props.contentOffset.y : 0,
525527
);
526528
this._scrollAnimatedValue.setOffset(
529+
// $FlowFixMe
527530
this.props.contentInset ? this.props.contentInset.top : 0,
528531
);
529532
this._stickyHeaderRefs = new Map();

Libraries/Lists/MetroListView.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ class MetroListView extends React.Component<Props, $FlowFixMeState> {
146146
/>
147147
);
148148
}
149-
_listRef: ListView;
149+
_listRef: ?ListView;
150150
_captureRef = ref => {
151151
this._listRef = ref;
152152
};

Libraries/StyleSheet/EdgeInsetsPropType.js

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,14 +12,12 @@
1212

1313
const PropTypes = require('prop-types');
1414

15-
const createStrictShapeTypeChecker = require('createStrictShapeTypeChecker');
16-
17-
const EdgeInsetsPropType = (createStrictShapeTypeChecker({
15+
const EdgeInsetsPropType = PropTypes.shape({
1816
top: PropTypes.number,
1917
left: PropTypes.number,
2018
bottom: PropTypes.number,
2119
right: PropTypes.number,
22-
}): ReactPropsCheckType & ReactPropsChainableTypeChecker);
20+
});
2321

2422
export type EdgeInsetsProp = {|
2523
+top: number,

Libraries/StyleSheet/PointPropType.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,7 @@
1212

1313
const PropTypes = require('prop-types');
1414

15-
const createStrictShapeTypeChecker = require('createStrictShapeTypeChecker');
16-
17-
const PointPropType = createStrictShapeTypeChecker({
15+
const PointPropType = PropTypes.shape({
1816
x: PropTypes.number,
1917
y: PropTypes.number,
2018
});

RNTester/js/ScrollViewExample.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ exports.examples = [
8080
title: string,
8181
additionalStyles: typeof StyleSheet,
8282
) {
83-
let _scrollView: ScrollView;
83+
let _scrollView: ?ScrollView;
8484
return (
8585
<View style={additionalStyles}>
8686
<Text style={styles.text}>{title}</Text>

0 commit comments

Comments
 (0)