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

rebind on to window.api #546

Closed
timelyportfolio opened this Issue Jan 26, 2017 · 3 comments

Comments

Projects
None yet
2 participants
@timelyportfolio

timelyportfolio commented Jan 26, 2017

I would like to attempt to use geojson.io in R with Shiny as a proof of concept. To accomplish this, I will need access to the gjIO.dispatch. I accomplished by adding after lines

d3.rebind(window.api, context.dispatch,'on');

to monitor change events.

Would this be something you would consider as a good addition to window.api, or should I just pursue in my fork? Thanks so much for this and all your incredible work, including your tweets!!

@tmcw

This comment has been minimized.

Show comment
Hide comment
@tmcw

tmcw Jan 31, 2017

Contributor

Hey Kent!

Thanks! And nice running into you on the hubs. Interesting proposal, I'm into it. Adding to my list of tweaks to make for the next geojson.io rebuild.

  • Tom
Contributor

tmcw commented Jan 31, 2017

Hey Kent!

Thanks! And nice running into you on the hubs. Interesting proposal, I'm into it. Adding to my list of tweaks to make for the next geojson.io rebuild.

  • Tom
@timelyportfolio

This comment has been minimized.

Show comment
Hide comment
@timelyportfolio

timelyportfolio Jan 31, 2017

@tmcw, 👍 great to hear and please let me know if I can help in any way. As additional reference, see geojsonio featured in this introductory mapedit post. Thanks again for everything!

timelyportfolio commented Jan 31, 2017

@tmcw, 👍 great to hear and please let me know if I can help in any way. As additional reference, see geojsonio featured in this introductory mapedit post. Thanks again for everything!

@tmcw tmcw closed this in 9766a80 Feb 2, 2017

@timelyportfolio

This comment has been minimized.

Show comment
Hide comment
@timelyportfolio

timelyportfolio commented Feb 2, 2017

cool, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment