Merge pull request #68 from martinRenou/improve_layout

Improve layout

Author: martinRenou

ipycanvas/canvas.py

@@ -177,8 +177,6 @@ def __init__(self, *args, **kwargs):
         self._buffers_cache = []
 
         super(Canvas, self).__init__(*args, **kwargs)
-        self.layout.width = str(self.size[0]) + 'px'
-        self.layout.height = str(self.size[1]) + 'px'
 
         self.on_msg(self._handle_frontend_event)
 
@@ -603,8 +601,6 @@ def __init__(self, n_canvases=3, *args, **kwargs):
         size = kwargs.get('size', (700, 500))
 
         super(MultiCanvas, self).__init__(*args, _canvases=[Canvas(size=size) for _ in range(n_canvases)], **kwargs)
-        self.layout.width = str(size[0]) + 'px'
-        self.layout.height = str(size[1]) + 'px'
 
     def __getitem__(self, key):
         """Access one of the Canvas instances."""

src/widget.ts

@@ -512,7 +512,15 @@ class MultiCanvasView extends DOMWidgetView {
     // @ts-ignore
     return this.create_child_view(canvasModel).then((canvasView: CanvasView) => {
       canvasView.el.style.zIndex = index;
-      canvasView.el.style.position = 'absolute';
+
+      if (index == 0) {
+        // This will enforce the container to respect the children size.
+        canvasView.el.style.position = 'relative';
+        canvasView.el.style.float = 'left';
+      } else {
+        canvasView.el.style.position = 'absolute';
+      }
+
       this.container.appendChild(canvasView.el);
 
       return canvasView;