diff options
author | Matthew Somerville <matthew@mysociety.org> | 2016-03-09 12:15:31 +0000 |
---|---|---|
committer | Matthew Somerville <matthew@mysociety.org> | 2016-03-09 12:15:31 +0000 |
commit | 17072b3dc80a0cfe285692494768d652ef1d1f4e (patch) | |
tree | 9aa71a84506baf164293d5516832f90bb1b2025f /web/js | |
parent | 2e0f8da2f9e1c99f87158fc439254fe8e4b42550 (diff) |
Remove auto-scrolling of sidebar on pin hover.
This was added along with pin/sidebar highlighting in b71d042e,
but has proved distracting/confusing.
Diffstat (limited to 'web/js')
-rw-r--r-- | web/js/map-OpenLayers.js | 20 |
1 files changed, 0 insertions, 20 deletions
diff --git a/web/js/map-OpenLayers.js b/web/js/map-OpenLayers.js index 6c99ee1f4..f2666787a 100644 --- a/web/js/map-OpenLayers.js +++ b/web/js/map-OpenLayers.js @@ -148,26 +148,6 @@ function fms_sidebar_highlight(problem_id) { if (typeof problem_id !== 'undefined') { var $a = $('.item-list--reports a[href$="' + problem_id + '"]'); $a.parent().addClass('hovered'); - - var a_top = $a.offset().top; - var a_height = $a.outerHeight(); - var viewport_top = $(window).scrollTop(); - var viewport_height = $(window).height(); - var header_height = $('.nav-wrapper-2').outerHeight(); - var shadow_height = $('.shadow-wrap').outerHeight(); - var shadow_top = $('.shadow-wrap').offset().top; - - if (a_top < viewport_top + header_height) { - // Top of item (at least) is above the bottom edge of the header - $('html, body').animate({ - scrollTop: a_top - header_height - 20 // little bit extra below header - }, 100); - } else if (a_top + a_height > shadow_top) { - // Bottom of item (at least) is below the top edge of the shadow - $('html, body').animate({ - scrollTop: a_top + a_height - viewport_height + shadow_height - }, 100); - } } else { $('.item-list--reports .hovered').removeClass('hovered'); } |