echarts icon indicating copy to clipboard operation
echarts copied to clipboard

fix(toolbox): use default `boundaryGap` when switching the line & bar series by the `toolbox` component

Open zhaxnb opened this issue 2 years ago • 3 comments

…ng default boundaryGap configuration.

Brief Information

This pull request is in the type of:

  • [x] bug fixing
  • [ ] new feature
  • [ ] others

What does this PR do?

Fixed using the toolbox to change line or bar charts but without using default boundaryGap configuration.

Fixed issues

Details

Before: What was the problem?

using the toolbox to change line or bar charts but without using default boundaryGap configuration.

After: How does it behave after the fixing?

start image before image after image

change image image

official website demo image image

official demo modify xAxis->boundaryGap: fasle to boundaryGap: true, click run then click switch to line chart.

Document Info

One of the following should be checked.

  • [x] This PR doesn't relate to document changes
  • [ ] The document should be updated later
  • [ ] The document changes have been made in apache/echarts-doc#xxx

Misc

ZRender Changes

  • [ ] This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs

N.A.

Others

Merging options

  • [ ] Please squash the commits into a single one when merging.

Other information

zhaxnb avatar Jan 15 '24 02:01 zhaxnb

Thanks for your contribution! The community will review it ASAP. In the meanwhile, please checkout the coding standard and Wiki about How to make a pull request.

echarts-bot[bot] avatar Jan 15 '24 02:01 echarts-bot[bot]

last pr #19428 is broken. this is new.

zhaxnb avatar Jan 15 '24 02:01 zhaxnb

The changes brought by this PR can be previewed at: https://echarts.apache.org/examples/editor?version=PR-19509@89007be

github-actions[bot] avatar Jan 15 '24 10:01 github-actions[bot]