From ea46e5dd57cd8bd0c09e1a302f5a5b0b590d1a90 Mon Sep 17 00:00:00 2001 From: Sam Date: Mon, 27 Jun 2016 17:22:24 +1000 Subject: [PATCH] UX: add minimum height for zoomed composer on mobile --- app/assets/javascripts/discourse/lib/safari-hacks.js.es6 | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/assets/javascripts/discourse/lib/safari-hacks.js.es6 b/app/assets/javascripts/discourse/lib/safari-hacks.js.es6 index 888bf34fc6..1aa215f534 100644 --- a/app/assets/javascripts/discourse/lib/safari-hacks.js.es6 +++ b/app/assets/javascripts/discourse/lib/safari-hacks.js.es6 @@ -73,7 +73,8 @@ function positioningWorkaround($fixedElement) { fixedElement.style.top = '0px'; - fixedElement.style.height = parseInt(window.innerHeight*0.6) + "px"; + const height = Math.max(parseInt(window.innerHeight*0.6), 350); + fixedElement.style.height = height + "px"; // I used to do this, but it seems like we don't need to with position // fixed