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

Panning with Context Menu #584

Closed
ajayre opened this issue Aug 16, 2022 · 1 comment
Closed

Panning with Context Menu #584

ajayre opened this issue Aug 16, 2022 · 1 comment

Comments

@ajayre
Copy link

ajayre commented Aug 16, 2022

I think panning should only be with a left mouse click, not right. When a context menu is being used and panning is enabled then right-clicking causes the chart to jump around. This is a problem if the point of the context menu is to place a markers at the location that was clicked.

I've fixed this in my copy by changing the Chart.Chart_PointerDown function to add a bool parameter indicating if the left mouse button was used, but that breaks backwards compatibility so I don't have a suitable patch to submit.

beto-rodriguez added a commit that referenced this issue Sep 1, 2022
@beto-rodriguez
Copy link
Owner

Thanks for the report, this is fixed with the referenced commit and also with #581

This will be supported ion the next version of the library.

@beto-rodriguez beto-rodriguez modified the milestone: v2.1 Sep 1, 2022
@beto-rodriguez beto-rodriguez mentioned this issue Sep 5, 2022
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

No branches or pull requests

2 participants