-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
paper.tools is never updated (wanting to use multiple tools for same canvas) #27
Comments
Yes this is a known issue. Only one tool is supported at the moment. This will be worked on soon. |
OK, can I help with this case? I feel like commenting out the paper = scope line would lead to a lot of other problems. Any pointers? |
…ve lists and references in the PaperScope (Project, View, Tool), so they can share functionality (#initialize(), #activate(), #remove()), and add support for multiple tools. Closes #27
…ve lists and references in the PaperScope (Project, View, Tool), so they can share functionality (#initialize(), #activate(), #remove()), and add support for multiple tools. Closes #27
How do I draw shapes so that when I press the left mouse button and move, I show the border / contour of a possible future figure? |
tried to start the commented code, but it does not work. I need it to work, as in the code below, but with several tools.
|
figured out, but now another problem. It is impossible to activate the tool.
|
Great job on this so far!
I was wanting to include multiple tools on the same canvas (draw a rectangle, circle, triangle, etc depending on the user pressing a button on the page outside of the canvas). I assumed the way to accomplish this would be something like:
This tools array never gets updated in code. Maybe there is a different way to accomplish what I am describing? Either way, I would expect paper.tools to be updated. https://github.com/paperjs/paper.js/blob/master/src/core/PaperScript.js#L157
But that only works if I comment out the line
It seems to work, but I am sure this is not the right way to go about it and it might cause other problems. Any advice?
The text was updated successfully, but these errors were encountered: