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

[ XY Plot Time Series ] - Zooming beyond the edge of the chart functionality #221

Open
5 of 6 tasks
JayWebz opened this issue May 20, 2021 · 6 comments
Open
5 of 6 tasks
Labels
Blocked Identifies a task that is stuck due to a limitation beyond dev control bug Something isn't working usability Improvements to usability of existing features

Comments

@JayWebz
Copy link
Contributor

JayWebz commented May 20, 2021

History

From #187 -

zooming beyond the edge of the chart is issue addressed on their forum https://canvasjs.com/forums/topic/problem-zooming-to-the-end-of-a-chart/. It seems that they do not consider it an issue with priority enough to update yet. The current solution is to hold down the mouse and continue trying to line up the zoom with the edge of the chart as the selection is still active. The solution https://canvasjs.com/forums/topic/hard-to-zoom-around-corners-of-graph/ allows me to add padding to the edge of the chart, however that is not desired, but currently implemented.

Goal

Remove current padding solution entirely, allow for zoomed range to be visible when dragging mouse beyond confines of the visual component.

Steps

  • Get account access to be able to post on the canvasjs.com forums.
  • Find their github and post an issue there.
  • See if there are packages built on top of canvasjs that solve for issue.
  • See if I can update the code directly to allow for the zoom to over reach. (Not to be spent too long on).

Short-term Solutions

  • Reduce padding value to 1%
  • Add an indication to the x-axis that indicates the current time. "NOW". Possibly date/time stamp.
@JayWebz JayWebz added bug Something isn't working usability Improvements to usability of existing features labels May 20, 2021
@JayWebz JayWebz added this to the v1.8.0 milestone May 20, 2021
@JayWebz
Copy link
Contributor Author

JayWebz commented May 20, 2021

Was able to post on their forum. Apparently UCSD email doesn't like the reset password email that comes from canvasjs. I re-registered with a gmail address and had no problem. I was able to post in the chat asking if the issue has any priority at this stage and expressing what the expected behavior should be. https://canvasjs.com/forums/topic/problem-zooming-to-the-end-of-a-chart/page/2/#post-34380.

canvasjs doesn't have a public github repo and all chat is funneled through their website forum.

@JayWebz
Copy link
Contributor Author

JayWebz commented May 24, 2021

a canvasjs representative responded to me today. They provided a snippet of a solution to a problem that doesn’t quite resemble the current issue. Responded with a video and some clarification, check back in tomorrow. https://canvasjs.com/forums/topic/problem-zooming-to-the-end-of-a-chart/page/2/#post-34424

@JayWebz
Copy link
Contributor Author

JayWebz commented May 26, 2021

representative responded with

Mouse events are captured only when mouse is on top of the chart. Any mouse event fired outside of the chart cannot be captured. Hence, it is not possible to trigger the zoom when mouse event is fired out of the chart as of now.

However, we will reconsider this behavior in future releases.

Going to suggest if an optional function can be passed into canvas js that contains the event listener so that the context outside of the react component is understood.

@JayWebz
Copy link
Contributor Author

JayWebz commented May 26, 2021

Reduced padding down from 5% to 1% - #232

@JayWebz
Copy link
Contributor Author

JayWebz commented Jun 8, 2021

I responded to the representative asking if its possible to set up an optional function prop to be passed to canvasjs that can help manage tracking the mouse location beyond the confines of the canvasjs component. https://canvasjs.com/forums/topic/problem-zooming-to-the-end-of-a-chart/page/2/#post-34561

@JayWebz
Copy link
Contributor Author

JayWebz commented Jun 16, 2021

representative responded with

Currently, the chart component has its own zoom and pan functionality, and cannot consume any events fired from a parent component that could trigger the zoom event.

@JayWebz JayWebz added the Blocked Identifies a task that is stuck due to a limitation beyond dev control label Oct 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocked Identifies a task that is stuck due to a limitation beyond dev control bug Something isn't working usability Improvements to usability of existing features
Projects
None yet
Development

No branches or pull requests

1 participant