Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

allow clearing of a single renderlet or all renderlets #779

Closed
wants to merge 1 commit into from

Conversation

davisford
Copy link
Contributor

See #776 for rationale

This adds two new functions on chart:

var renderletFn = function (chart) { /* do something with chart */ };
// add a renderlet
chart.renderlet(renderletFn);
// clear it
chart.clearRenderlet(renderletFn);
// clear all renderlets
chart.clearRenderlets();

@r4j4h
Copy link
Contributor

r4j4h commented Jan 5, 2015

I think gordon's point about the real solution ultimately using d3.dispatch is good to echo here, and I 👍 to having this available in the meantime.

@gordonwoodhull gordonwoodhull added this to the v2.0 milestone Jan 17, 2015
@gordonwoodhull
Copy link
Contributor

#776 covers this in a more general way, merged in beta.2. Thanks for bringing this up, @davisford!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants