From d26b320ab7446b208bd1fbdb009a1811957fdf22 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Sat, 4 Jul 2020 02:22:10 +0200 Subject: [PATCH] keyboard: Fix setting height in portrait orientation get_preferred_height() returns both the minimum and natural height, not a single size. Math.min() doesn't handle that and returns NaN, whoops. https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/2349 --- js/ui/keyboard.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/js/ui/keyboard.js b/js/ui/keyboard.js index e8a79f507..a3a30c023 100644 --- a/js/ui/keyboard.js +++ b/js/ui/keyboard.js @@ -1609,7 +1609,8 @@ class Keyboard extends St.BoxLayout { * we allow the OSK being smaller than 1/3rd of the monitor height * there. */ - this.height = Math.min(maxHeight, this.get_preferred_height(monitor.width)); + const [, natHeight] = this.get_preferred_height(monitor.width); + this.height = Math.min(maxHeight, natHeight); } }