fix(label): fix labelLine default style when normal state not show but other state show

[linghaoSu]

Brief Information

This pull request is in the type of:

• bug fixing
• new feature
• others

What does this PR do?

fix labelLine style not follow label style when normal state show: false, other state show: true;

Fixed issues

• fix [Bug] 在使用pie南丁格尔玫瑰图的时候动态隐藏labelLine，线的颜色一直是黑色的，无法使用label颜色 #18680

Details

Before: What was the problem?

when set normal state label show is false, and other state(such as emphasis) state label show: true

labelLine style cannot inherit sector color, and always #000

After: How does it behave after the fixing?

although normal state label is show: false, but once any state has show: true, labelStyle will inhreit sector style

Document Info

One of the following should be checked.

• 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

[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.

[github-actions]

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

[Ovilia]

I think the logic is all right but I'd prefer the way #19180 because setLabelLineState is not a frequent operation and thus checking show in all states can be ignored.

However, this PR gives a very solid test for it so I would also like to merge it. Can you revert the changes of src and pull & merge from the master and submit again? Thanks!

[linghaoSu]

I think the logic is all right but I'd prefer the way #19180 because setLabelLineState is not a frequent operation and thus checking show in all states can be ignored.

However, this PR gives a very solid test for it so I would also like to merge it. Can you revert the changes of src and pull & merge from the master and submit again? Thanks!

I apologize for the delayed response. I am pleased to see that the bug has already been resolved, and I am willing to revert the code in the src directory.

[Ovilia]

Thanks for co-author the PR!

[echarts-bot]

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