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

Transform inclusion/composition. #3

Closed
mbostock opened this issue Sep 30, 2010 · 1 comment
Closed

Transform inclusion/composition. #3

mbostock opened this issue Sep 30, 2010 · 1 comment

Comments

@mbostock
Copy link
Member

There should be a way to define a transform and include it in another transform. Perhaps: a.include(b).

@mbostock
Copy link
Member Author

On another note, we should make it more explicit that calling apply on a transform always applies the root transform, rather than the current scope. Or even prevent such mistakes by only defining the apply method on root transforms.

Alternatively, it might be possible to apply a subset of a transform, but I suspect that this will be confusing as the transform will be applied in the global scope, rather than in the context of how it was defined.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

1 participant