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 event handlers to the Draw plugin #2000

Conversation

hansthen
Copy link
Collaborator

@hansthen hansthen commented Oct 5, 2024

This makes the draw plugin more dynamic. It allows the user to add JsCode event handlers to the generated layers.

Also removed duplicate actions for the "draw:created" event.

@hansthen hansthen requested a review from Conengmo October 5, 2024 13:37
@hansthen hansthen force-pushed the add_event_handlers_to_drawn_layers_in_draw_plugin branch from 73b616a to a2c9292 Compare October 5, 2024 14:02
This makes the draw plugin more dynamic. It allows the user
to add `JsCode` event handlers to the generated layers.
@hansthen hansthen force-pushed the add_event_handlers_to_drawn_layers_in_draw_plugin branch from 3e53b86 to c787898 Compare October 5, 2024 14:08
@hansthen hansthen changed the title Add event handlers to the draw plugin Add event handlers to the Draw plugin Oct 6, 2024
Copy link
Member

@Conengmo Conengmo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Two small comments, but the functionality is good!

requirements.txt Outdated
numpy
numpy==1.*
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please include this in a separate PR (if necessary)

"""
'''
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm pretty sure Black uses double quotes? Why include this change?

@hansthen hansthen merged commit 5a7a173 into python-visualization:main Oct 18, 2024
8 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants