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

Make the MultiCanvas one canvas element in the DOM structure? #112

Closed
martinRenou opened this issue Jul 25, 2020 · 0 comments · Fixed by #113
Closed

Make the MultiCanvas one canvas element in the DOM structure? #112

martinRenou opened this issue Jul 25, 2020 · 0 comments · Fixed by #113

Comments

@martinRenou
Copy link
Collaborator

martinRenou commented Jul 25, 2020

Instead of stacking each Canvas view on top of each other to build the MultiCanvas in the DOM, we might want to use one canvas element on which we draw all layers from the CanvasModels.

This would have some benefits:

  • The DOM structure will be simpler
  • The MultiCanvas sizing/layout will behave exactly like a normal Canvas
  • We don't need to create multiple Canvas views, we can use offscreen canvases

We might loose some performances though. As when one canvas updates, we need to redraw all layers using drawImage.

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 a pull request may close this issue.

1 participant