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

Make data point to show over radar chart #952

Closed
netil opened this issue Jun 27, 2019 · 1 comment
Closed

Make data point to show over radar chart #952

netil opened this issue Jun 27, 2019 · 1 comment

Comments

@netil
Copy link
Member

netil commented Jun 27, 2019

Description

The current implementation of radar looks like:
image

The datapoints are part of <g class='bb-chart-lines'> and the radar node appear after, which makes data points to stay behind the radar element.

Usually data points should stay over the chart area, so in this case the current(left) should be moved as the follows(right).
image

As a result :
image

@netil netil added the bug label Jun 27, 2019
@netil netil self-assigned this Jun 27, 2019
@netil netil added the patch label Jun 27, 2019
@netil netil closed this as completed in 4db457d Jun 27, 2019
netil pushed a commit that referenced this issue Jun 28, 2019
## [1.9.3](1.9.2...1.9.3) (2019-06-28)

### Bug Fixes

* **color:** Fix on color.threshold.values handling ([841b316](841b316)), closes [#950](#950)
* **radar:** Fix radar positioning ([612d93f](612d93f)), closes [#953](#953)
* **radar:** Make data points stay over radar ([4db457d](4db457d)), closes [#952](#952)
@netil
Copy link
Member Author

netil commented Jun 28, 2019

🎉 This issue has been resolved in version 1.9.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@netil netil added the released label Jun 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant