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

Add radar chart implementation #386

Merged
merged 1 commit into from
Aug 10, 2018

Conversation

loictro
Copy link
Contributor

@loictro loictro commented Aug 9, 2018

It adds the Radar chart feature.
Most of the existing chart options should be well rendered.

  • The following option is specific:
Name Description
radarStyle standard (default),marker, filled

An example of Radar charts is added in the HTML demo file. (Slide 15 of charts).
The generated file is OOXML valid and seems to open well with Microsoft Powerpoint 2010 ;)

@gitbrent gitbrent self-assigned this Aug 10, 2018
@gitbrent gitbrent merged commit 678da54 into gitbrent:master Aug 10, 2018
@gitbrent gitbrent added this to the 2.3.0 milestone Aug 10, 2018
@gitbrent
Copy link
Owner

Great job @loictro !

Thanks for the contribution.

@gitbrent gitbrent self-requested a review August 10, 2018 03:53
gitbrent added a commit that referenced this pull request Aug 10, 2018
ericrockey pushed a commit to WeTransfer/PptxGenJS that referenced this pull request Mar 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants