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

[Feature] add borderRadius option to heatmap #16490

Closed
fmmattioni opened this issue Feb 10, 2022 · 4 comments
Closed

[Feature] add borderRadius option to heatmap #16490

fmmattioni opened this issue Feb 10, 2022 · 4 comments
Labels
difficulty: easy Issues that can be fixed more easily than the average. en This issue is in English new-feature

Comments

@fmmattioni
Copy link

What problem does this feature solve?

Taking this ECharts example https://echarts.apache.org/examples/en/editor.html?c=heatmap-cartesian, I was wondering if it would be somehow possible to apply a border-radius to each square of the heatmap. My goal was that make these squares appear as rounded elements, like that: https://9elements.com/blog/content/images/border-radius-1.png.

I have tried using the series-heatmap.itemStyle.borderRadius property, but that does not seem available.

Any ideas?

What does the proposed API look like?

I believe adding series-heatmap.itemStyle.borderRadius would solve the problem easily.

@echarts-bot echarts-bot bot added en This issue is in English pending We are not sure about whether this is a bug/new feature. waiting-for: community labels Feb 10, 2022
@pissang pissang added the difficulty: easy Issues that can be fixed more easily than the average. label Feb 11, 2022
@echarts-bot
Copy link

echarts-bot bot commented Feb 11, 2022

This issue is labeled with difficulty: easy.
@fmmattioni Would you like to debug it by yourself? This is a quicker way to get your problem fixed. Or you may wait for the community to fix.

Please have a look at How to debug ECharts if you'd like to give a try. 🤓

@pissang pissang removed pending We are not sure about whether this is a bug/new feature. waiting-for: community labels Feb 11, 2022
Rain120 added a commit to Rain120/echarts that referenced this issue Feb 11, 2022
pissang pushed a commit that referenced this issue Feb 15, 2022
* fix(chart): DefaultStatesMixinEmpasis -> DefaultStatesMixinEmphasis in util/types

* feat: heatmap itemStyle.borderRadius with #16490

* feat(test): format heatmap-geo

* feat(test): heatmap itemStyle.borderRadius

* feat(chart): borderRadius support data item; adjust RectShape type be the instance with graphic.Rect

* feat(test): Heatmap Border Radius by Each Item

* feat(heatmap): mktest for heatmap-border-radius

* feat(heatmap): get borderRadius value before ensureState set emphasis

* feat(test): rename heatmap-border-radius to heatmap-borderRadius

* feat(heatmap): remove useless temporary variable shape

* feat(test): format heatmap (geo, borderRadius)

* feat(heatmap): remove uesless import in HeatmapView

* fix(test): revert heatmap-geo.html

* fix(heatmap): merge type & restore lost type about Calendar
@paulkarpenko
Copy link

@Rain120 Came here looking for a way to add borderRadius to bar charts and found this. The prop is not included in barItemStyle.js and I haven't seen any examples showing rounded-corner bar charts. Can it be added to bar as well as headmap? Thank you! 🙏🏻

@pissang
Copy link
Contributor

pissang commented Mar 7, 2022

@paulkarpenko
Copy link

@pissang thank you! I see this in the latest package.

I'm using 4.9.0 which doesn't seem to support this, is that right?

@susiwen8 susiwen8 closed this as completed Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty: easy Issues that can be fixed more easily than the average. en This issue is in English new-feature
Projects
None yet
Development

No branches or pull requests

4 participants