🐛 hide full screen button when not useful (#4305)

owid · Dec 13, 2024 · 5a48228 · 5a48228
1 parent f319d16
commit 5a48228
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/packages/@ourworldindata/grapher/src/core/Grapher.tsx b/packages/@ourworldindata/grapher/src/core/Grapher.tsx
@@ -2419,9 +2419,10 @@ export class Grapher
     }
 
     @computed get hideFullScreenButton(): boolean {
-        const { windowInnerHeight } = this
-        if (!windowInnerHeight) return true
-        return this.windowInnerHeight! < this.frameBounds.height
+        // hide the full screen button if the full screen height
+        // is barely larger than the current chart height
+        const fullScreenHeight = this.windowInnerHeight!
+        return fullScreenHeight < this.frameBounds.height + 80
     }
 
     @computed private get availableWidth(): number {