Skip to content

exposing canvas and canvastmp in options#19

Open
w3sami wants to merge 1 commit intopwambach:masterfrom
w3sami:expose-canvas
Open

exposing canvas and canvastmp in options#19
w3sami wants to merge 1 commit intopwambach:masterfrom
w3sami:expose-canvas

Conversation

@w3sami
Copy link
Copy Markdown
Contributor

@w3sami w3sami commented Oct 29, 2015

so I can access in the controller using the painter. btw, I couldn't figure out, why there are two canvases, the tmp canvas can maybe be omitted from exposure?

@pwambach
Copy link
Copy Markdown
Owner

sorry for the delay, i don't think exposing the canvas in the options is a good way to do it, the options object should be used only for configuration. perhaps it is better to expose the canvas through an extra service. another option would be to expose it as a scope property or through a callback function.
what do you want to do with the canvas? get the image or also other stuff?

cheers phil

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