fix(app): apply padding to SampleView when it's at root

genome-spy · Mar 23, 2023 · 9f0f38c · 9f0f38c
1 parent 01217cd
commit 9f0f38c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/packages/app/src/sampleView/sampleView.js b/packages/app/src/sampleView/sampleView.js
@@ -683,6 +683,11 @@ export default class SampleView extends ContainerView {
             return;
         }
 
+        if (!this.parent) {
+            // Usually padding is applied by GridView, but if this is the root view, we need to apply it here
+            coords = coords.shrink(this.getPadding());
+        }
+
         context.pushView(this, coords);
 
         const cols = mapToPixelCoords(

diff --git a/packages/core/src/view/gridView.js b/packages/core/src/view/gridView.js
@@ -527,7 +527,11 @@ export default class GridView extends ContainerView {
             return;
         }
 
-        coords = coords.shrink(this.getPadding()); // TODO: Only applicable at view root
+        if (!this.parent) {
+            // Usually padding is applied by the parent GridView, but if this is the root view, we need to apply it here
+            coords = coords.shrink(this.getPadding());
+        }
+
         context.pushView(this, coords);
 
         const flexOpts = {