-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
scrollResponderZoomTo does not work in Android #9830
Comments
This is a limitation of the Android ScrollView. It doesn't have built-in support for zooming or multi-directional scrolling. |
Understood. However from the point of view of Arab, this is a problem that If an additional native component is needed, we should track that work On Sunday, October 9, 2016, Jake Murzy notifications@github.com wrote:
Blog: shaneosullivan.wordpress.com |
Agreed. I'm currently working on reimplementing UIScrollView in JavaScript. IMO this is ultimately the best way to achieve feature parity between native platforms. l will update here when if I make noteworthy progress. 🍺 |
This is definitely a good feature to have. We are trying to keep github issues focused on bug reports, though, and use Product Pains for feature requests, so I think we should close this issue. It's also somewhat stale and it's unclear at least to me how one would plausibly solve it (I suspect ScrollView in JS will have a lot of tradeoffs). But if someone had a PR for this that would be cool! |
@jmurzy Do you reimplemt the new one ? |
+1 |
2 similar comments
+1 |
+1 |
+1 please help us! that is a huge PROBLEM!!!! |
+1 |
1 similar comment
+1 |
+1 ! |
Issue Description
ScrollResponder.scrollResponderZoomTo only works in iOS. This prevents the programmatic control of zoom in ScrollViews.
It contains the code
if (Platform.OS === 'android') {
invariant('zoomToRect is not implemented');
}
Steps to Reproduce / Code Snippets
Simply call the scrollResponderZoomTo on any ScrollView component, e.g.
this.refs.scrollView.scrollResponderZoomTo({
x: 1,
y: 1,
width: 400,
height: 800,
animated: false,
});
Expected Results
It should zoom to the desired rectangle. However it throws an error with an invariant.
Additional Information
The text was updated successfully, but these errors were encountered: