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

Factor "insert" into it's own API #1322

Open
dethe opened this issue Feb 7, 2016 · 0 comments
Open

Factor "insert" into it's own API #1322

dethe opened this issue Feb 7, 2016 · 0 comments
Assignees

Comments

@dethe
Copy link
Member

dethe commented Feb 7, 2016

We currently insert new blocks from dragging, tapping, paste, and undo/redo. We have a lot of duplicated code (and bugs) because of this, and making changes to how it works (such as allowing wb-expressions to be dropped in with wb-steps by automatically wrapping them) has to be done in multiple places.

This is also some of our oldest and gnarliest code and long overdue for shining a light on. Ideally we can leverage the custom elements to have a more OO approach for this instead of the current mess of nested if/else.

Related to #1320

@dethe dethe self-assigned this Feb 7, 2016
@dethe dethe added this to the Cut/Copy/Paste milestone Feb 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant