-
Notifications
You must be signed in to change notification settings - Fork 24.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add viewConfig to RCTSlider Component, fix #10158 #10238
Conversation
@leeight updated the pull request - view changes |
1 similar comment
@leeight updated the pull request - view changes |
this._component.setNativeProps( | ||
this._propsAnimated.__getAnimatedValue() | ||
); | ||
if (this._component.viewConfig != null) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you throw an error here instead of just dropping the change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@javache Done.
@leeight updated the pull request - view changes |
@facebook-github-bot shipit |
Thanks for importing. If you are a Facebook employee, you can view this diff on Phabricator. |
Summary: And check componet.viewConfig before call component.setNativeProps method. FIX #10158 Closes facebook/react-native#10238 Differential Revision: D4008640 Pulled By: javache fbshipit-source-id: 1bf9b34c0e56f1ffbcd31ae64b3cbafe77a9a1fd
Summary: And check componet.viewConfig before call component.setNativeProps method. FIX #10158 Closes facebook/react-native#10238 Differential Revision: D4008640 Pulled By: javache fbshipit-source-id: 1bf9b34c0e56f1ffbcd31ae64b3cbafe77a9a1fd
Summary: And check componet.viewConfig before call component.setNativeProps method. FIX #10158 Closes facebook/react-native#10238 Differential Revision: D4008640 Pulled By: javache fbshipit-source-id: 1bf9b34c0e56f1ffbcd31ae64b3cbafe77a9a1fd
And check componet.viewConfig before call component.setNativeProps method.
FIX #10158