From 518467617ebe40f6a28eba95be5ab5b9148e7bf8 Mon Sep 17 00:00:00 2001 From: Stefan Breunig Date: Thu, 14 Mar 2013 12:56:39 +0100 Subject: [PATCH] Instead of determining the height of the sidebar and fixing it to that, this patch uses the CSS pointer-events directive to make the area between the side bar and the map status usable. This should avoid cutting off the sidebar and force users to scroll even if there is space available but still allow to interact with that area. Please report if there are any problems with this. fix #467, fix #461 --- code/boot.js | 5 ----- style.css | 2 ++ 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/code/boot.js b/code/boot.js index f3ffd98e..6be827c4 100644 --- a/code/boot.js +++ b/code/boot.js @@ -355,11 +355,6 @@ function boot() { if(window.bootPlugins) $.each(window.bootPlugins, function(ind, ref) { ref(); }); - // sidebar is now at final height. Adjust scrollwrapper so scrolling - // is possible for small screens and it doesn’t block the area below - // it. - $('#scrollwrapper').css('max-height', ($('#sidebar').get(0).scrollHeight+3) + 'px'); - window.runOnSmartphonesAfterBoot(); // workaround for #129. Not sure why this is required. diff --git a/style.css b/style.css index f0bbd536..30cf0b96 100644 --- a/style.css +++ b/style.css @@ -21,6 +21,7 @@ body { width: 340px; bottom: 45px; z-index: 1001; + pointer-events: none; } #sidebar { @@ -34,6 +35,7 @@ body { overflow-y:scroll; overflow-x:hidden; z-index: 3000; + pointer-events: auto; } #sidebartoggle {