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
Append dom to workspace #961
Conversation
Updated Blockly.Xml.domToWorkspace to load previously saved "blocks" below the existing blocks
The function align the newly loaded blocks below the exisiting blocks
Add function appendDomToWorkspace to compute offset of x and y and call domToWorkspace; modified domToWorkspace to accept offset value; modified test_domToWorkspace to test the new parameter; add a test_appendDomToWorkspace to xml_test.js
…e_domToWorkspace_QN # Conflicts: # tests/jsunit/xml_test.js
domToWorkspace return array of new block ids appendDomToWorkspace call domToWorkspace then move the blocks One test added to workspace_svg_test
We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for the commit author(s). If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google. |
This is a copy (with additional comments) of PR #822 (and also #961) by @qnoirhomme with unrelated files removed. See #822 for full review.
This PR still has numerous unrelated files. I've made my own PR of the relevant code in #962. |
This is a pull request related to the previous pull request #822 .
A new function appendDomToWorkspace is created which append new blocks below any existing blocks in the workspace. domToWorkspace is modified to return the list of new block ids.