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

stock tools -- annotation miss its position when print and export as image #10605

Closed
wy1713 opened this issue Apr 23, 2019 · 2 comments · Fixed by #10619
Closed

stock tools -- annotation miss its position when print and export as image #10605

wy1713 opened this issue Apr 23, 2019 · 2 comments · Fixed by #10619

Comments

@wy1713
Copy link

wy1713 commented Apr 23, 2019

Expected behaviour

With stock tools user can make an text, circle, square annotation on the highstock chart. The annotation position should be exactly the same place when it is drawn on the chart when using the exporting module to print the chart, export the chart to different types.

Actual behaviour

however when exporting the chart, or print, the annotation missed it's original position of the chart and locationed somewhere else

Live demo with steps to reproduce

https://jsfiddle.net/wzLfrhgs/
see attachment, I draw an small square on the highest value. but when I export as an png, it is miss aligned.

Product version

Screenshot (21)

lastest

Affected browser(s)

chrome

@TorsteinHonsi
Copy link
Collaborator

@sebastianbochan

@pawelfus pawelfus self-assigned this Apr 24, 2019
pawelfus added a commit that referenced this issue Apr 25, 2019
…) added using stock-tools were misplaced in exported chart.
TorsteinHonsi pushed a commit that referenced this issue Apr 25, 2019
…) added using stock-tools were misplaced in exported chart.
@ghost ghost removed the Status: Pending review label Apr 25, 2019
@TorsteinHonsi
Copy link
Collaborator

TorsteinHonsi commented Apr 26, 2019

@pawelfus The test fails in Firefox, can you fix? Precision issues. I'll preliminary skip it so we can continue experimenting with CI.

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

Successfully merging a pull request may close this issue.

3 participants