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

d2b add showlabels and click event #71

Closed
dockstreet opened this issue Jun 2, 2018 · 6 comments
Closed

d2b add showlabels and click event #71

dockstreet opened this issue Jun 2, 2018 · 6 comments

Comments

@dockstreet
Copy link

@dockstreet dockstreet commented Jun 2, 2018

Hi,

Could the sunburstChartConfig be a parameter to allow for showLabels and a click event to observe as the non-d2b sunburstR?

@dockstreet dockstreet changed the title d2b add showlabels d2b add showlabels and click event Jun 3, 2018
@timelyportfolio
Copy link
Owner

@timelyportfolio timelyportfolio commented Jun 7, 2018

@dockstreet, are these options with d2b? I see showLabels for svgSunburst, but I don't think there is an easy accessor for showLabels with chartSunburst(). I think there is a way to change within the generator, but I don't honestly remember. Sorry if I am misunderstanding.

@dockstreet
Copy link
Author

@dockstreet dockstreet commented Jun 8, 2018

@timelyportfolio I saw an example here
d2bjs/d2b#13
and in this fiddle:
https://jsfiddle.net/3cyeddyp/4/

does this help explain a bit of what I'm trying to achieve by showing the labels?

Plus - is there a way to add an observable click event and mouse over as the non-d2b sunburst?

@jtrecenti
Copy link

@jtrecenti jtrecenti commented Aug 17, 2018

+1 to click event and mouse over as the non-d2b sunburst

timelyportfolio added a commit that referenced this issue Jul 6, 2020
@timelyportfolio
Copy link
Owner

@timelyportfolio timelyportfolio commented Jul 6, 2020

Sorry, I should have added this in the last version. I forgot. Added showLabels argument to sund2b and just submitted to CRAN. Thanks for the issue.

@timelyportfolio
Copy link
Owner

@timelyportfolio timelyportfolio commented Jul 11, 2020

Going to close for now, but please reopen if you would like to see better event integration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.