Feat: new selectedMode: series and select.disabled

[susiwen8]

Brief Information

This pull request is in the type of:

• bug fixing
• new feature
• others

What does this PR do?

• Since 5.0, ECharts supports item select, It would be nice to have selectable option for each item. The idea came from Support that some region selectable:false in geo. #14712

Also add series for selectedMode, it's would make whole series selected by on click.

Fix issue

• Close Support that some region selectable:false in geo. #14712

Details

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

bar2.html
geo-map.html

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.

[pissang]

Sorry for the late review. Seems to be a very promise feature. Take a quick look and here is one suggestion:

Could selectable be in the root instead of under itemStyle. It's not a style related option and not all series use itemStyle. I think there are two options:

data: [{ selectable: false }]

Or

data: [ select: { disabled: true } ]

This can be designed together with #14952 . Emphasis state can also be disabled:

emphasis: { disabled: true }

Or

hoverEmphasis: false

[susiwen8]

@pissang I prefer first one data: [{ selectable: false }]
However the second proposal can provide more fine grade control for item selected state, such as specify select color for single data item

[pissang]

@susiwen8 We aready have select for each data item. Where we can sepcify selected color and other styles. From my side I prefer using select.disabled. And it will be more consistent with emphasis.disabled.

[susiwen8]

@pissang Please check again

[pissang]

It's more common to use enabled / disabled instead of enable / disable. For example in highcharts https://api.highcharts.com/highcharts/boost.enabled

[susiwen8]

@pissang Oops I was meant to use disabled

[pissang]

LGTM now. @plainheart any more comments?

[pissang]

@susiwen8 BTW it's not correct to use chore prefix in the commit message here. chore means updating grunt tasks etc; no production code change

[echarts-bot]

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