1491668b35
Summary: The previous implementation of ColorPropType was very hacky as it used `ReactPropTypes.oneOfType([colorValidator, ReactPropTypes.number])`. It turns out that oneOfType also accepts arbitrary functions instead of a type, but doesn't display any of the error message. In this diff I properly implement isRequired (sadly we don't export `createChainableTypeChecker` in ReactPropTypes) and provide a lot more context that we have. I copy and pasted the way we displayed this context from the existing checkers. **Test Plan** When doing .isRequired and do not provide the value: ![simulator screen shot feb 1 2016 9 56 00 am](https://cloud.githubusercontent.com/assets/197597/12726239/61243f88-c8cb-11e5-889b-6594ffd85973.png) When providing a bad value: ![simulator screen shot feb 1 2016 10 01 25 am](https://cloud.githubusercontent.com/assets/197597/12726244/6e80aa36-c8cb-11e5-9bd3-a8637de75496.png) Closes https://github.com/facebook/react-native/pull/5671 Reviewed By: svcscm Differential Revision: D2886760 Pulled By: vjeux fb-gh-sync-id: d6be42b5768fca5463fe80fe4b144506d21b0832 |
||
---|---|---|
.. | ||
__tests__ | ||
ColorPropType.js | ||
EdgeInsetsPropType.js | ||
LayoutPropTypes.js | ||
PointPropType.js | ||
StyleSheet.js | ||
StyleSheetPropType.js | ||
StyleSheetRegistry.js | ||
StyleSheetTypes.js | ||
StyleSheetValidation.js | ||
TransformPropTypes.js | ||
flattenStyle.js | ||
normalizeColor.js | ||
processColor.js | ||
processTransform.js | ||
styleDiffer.js |