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

SVG and Canvas have different results for "Candlestick shanghai index" chart in echarts example #9571

Closed
wmjcom opened this issue Dec 12, 2018 · 3 comments

Comments

@wmjcom
Copy link

wmjcom commented Dec 12, 2018

I test "Candlestick shanghai index" chart in echarts example. This is the url "https://ecomfe.github.io/echarts-examples/public/editor.html?c=candlestick-sh". I click the "Download" button to save the html file into local. Then I compare the chart to render on canvas and svg.
The first result is the default html. It looks good.
image

I only make below changes to support svg in the html.
var myChart = echarts.init(dom, null, {renderer: 'svg'});
image

The svg chart has three issues different to canvas chart.
(1) Title "上证指数" is out of top range.
(2) The text data for xAxis are shown in the xAxis line. And the text data are not same in both results.
(3) The value "2445" is out of the ball.

Can anyone give some suggestions? Thanks.

@Ovilia
Copy link
Contributor

Ovilia commented Dec 13, 2018

Hi. What's the version of your SVG ECharts? I tried with the latest version and it's the same as canvas result.

@wmjcom
Copy link
Author

wmjcom commented Dec 17, 2018

      Hi. What's the version of your SVG ECharts? I tried with the latest version and it's the same as canvas result.

@Ovilia
I just download the example from "https://ecomfe.github.io/echarts-examples/public/editor.html?c=candlestick-sh". It is using "<script type="text/javascript" src="http://echarts.baidu.com/gallery/vendors/echarts/echarts.min.js"></script>".
I only make below changes to support svg in the html.
var myChart = echarts.init(dom, null, {renderer: 'svg'});
It still has these issues.
Are there anything else do I need change?

@Ovilia
Copy link
Contributor

Ovilia commented Jan 29, 2019

Close due to cannot reproduce.

@Ovilia Ovilia closed this as completed Jan 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants