Skip to content
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 a way to disable mouse move, pan or zoom #164

Merged
merged 2 commits into from
Nov 4, 2016

Conversation

shprink
Copy link
Contributor

@shprink shprink commented Nov 4, 2016

Closes #163

```
enabling `zoom` and `pan` to `true`

By default the chart is not focused. meaning. you have to click on the chart to get focus, and only then will the scroll events trigger a zoom.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing the docs too, can you remove

- By default the chart is not focused. meaning. you have to click on the chart to get focus, and only then will the scroll events trigger a zoom.

@shprink
Copy link
Contributor Author

shprink commented Nov 4, 2016

done

@rrag rrag merged commit 2a5e326 into rrag:master Nov 4, 2016
@rrag
Copy link
Owner

rrag commented Nov 4, 2016

Thank you. will rebuild and publish the docs site later

@rrag
Copy link
Owner

rrag commented Nov 8, 2016

Thanks for the PR

published new beta version, install again as npm i -S react-stockcharts@next

I had forgotten about the zoom capabilities by click & drag of the x and y axis, fixed that also, look at zoomEnabled prop in XAxis, YAxis in CandleStickChartWithZoomPan.jsx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants