fix(marker): add missing type for symbolOffset, fix symbolOffset & symbolKeepAspect doesn't work in markPoint.

[plainheart]

Brief Information

This pull request is in the type of:

• bug fixing
• new feature
• others

What does this PR do?

Enhance #14375

• add missing value type string | number to symbolOffset
• add missing return type string | number to SymbolOffsetCallback
• fix symbolOffset & symbolKeepAspect doesn't work in markPoint
• fix symbolOffset type issue in SilderTimelineView
• extract normalizeSymbolSize & normalizeSymbolOffset to the common symbol util.

Fixed issues

Details

Before: What was the problem?

After: How is it fixed in this PR?

Misc

• The API has been changed (apache/echarts-doc#xxx).
• This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs

NA.

Others

Merging options

• Please squash the commits into a single one when merge.

Other information

[echarts-bot]

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.

The pull request is marked to be PR: author is committer because you are a committer of this project.

[plainheart]

Currently, the logic about parsing symbolOffset with numberUtil.parsePercent and symbolSize is used in many places. I'm thinking to extract it into a common helper to reduce redundant code and bundled size.

[PiterYang]

when can publish a new version for this merge ?

[Ovilia]

LGTM

[Ovilia]

when can publish a new version for this merge ?

@PiterYang This PR is set to be in 5.2.0, which should be released in about a month.

[packjie]

I encountered the same problem. After upgrading the version, markpoint fell down. After checking, I found that the value of symboloffset did not work.

After reading this post, I changed the original file
1、node_ modules/echarts/lib/component/marker/MarkPointView.js


The problem was solved and the chart returned to normal

[echarts-bot]

Congratulations! Your PR has been merged. Thanks for your contribution! 👍

[osgregs]

ty, waiting new version.

[Flourad]

求问下这个特性什么时候能发布呀？

[alanhu151]

when can publish a new version for this merge ?

@PiterYang This PR is set to be in 5.2.0, which should be released in about a month.

When the 5.2.0 will be released? checked everyday for months. :(