Skip to content

Add padAngle option for pie/donut#2669

Merged
kt3k merged 6 commits intoc3js:masterfrom
cogniteev:feat/pad_angle
Aug 9, 2019
Merged

Add padAngle option for pie/donut#2669
kt3k merged 6 commits intoc3js:masterfrom
cogniteev:feat/pad_angle

Conversation

@panthony
Copy link
Contributor

@panthony panthony commented Aug 7, 2019

This PR adds:

  • padAngle option for both pie & donut
  • a an API method to get/set padAngle
  • add documentation in reference

It supersede #1921 and closes #1920

I was not quite sure how to expose the API, should we duplicate under both donut & pie key? Something else?

@codecov-io
Copy link

codecov-io commented Aug 8, 2019

Codecov Report

Merging #2669 into master will increase coverage by 0.16%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2669      +/-   ##
==========================================
+ Coverage   80.13%   80.29%   +0.16%     
==========================================
  Files          57       59       +2     
  Lines        4520     4538      +18     
==========================================
+ Hits         3622     3644      +22     
+ Misses        898      894       -4
Impacted Files Coverage Δ
src/config.js 95.83% <ø> (ø) ⬆️
src/api.donut.js 100% <100%> (ø)
src/api.pie.js 100% <100%> (ø)
src/arc.js 75.17% <100%> (+1.21%) ⬆️
src/api.chart.js 19.23% <0%> (+7.69%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a4c971c...d35ba77. Read the comment docs.

Copy link
Member

@kt3k kt3k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice feature to have 👍 LGTM!

スクリーンショット 2019-08-09 15 37 25

@kt3k kt3k merged commit 20dff29 into c3js:master Aug 9, 2019
@panthony panthony deleted the feat/pad_angle branch August 9, 2019 06:53
beninsocrata pushed a commit to socrata/c3 that referenced this pull request Oct 31, 2019
* add pie.padAngle and donut.padAngle options

* add API to get/set padAngle for both pie and donut

* add pie.padAngle and donut.padAngle in reference doc

* add padAngle option in donut sample

* bind getPadAngle method to self

* use chart main instead of global D3 to select chart's elements
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

new padAngle options for donut & pie type

3 participants