From f1d2855dcab4c0a0a7c3ad5b279a1b44c33a3b1a Mon Sep 17 00:00:00 2001
From: alexcojocaru <alexcojocaru@gmail.com>
Date: Wed, 4 Nov 2020 20:08:31 -0800
Subject: [PATCH] Add back the collapse/expand and resize improvements

---
 js/plugin/Heightgraph.js | 52 ++++++++++++++++++++++++++++++++++++++--
 1 file changed, 50 insertions(+), 2 deletions(-)

diff --git a/js/plugin/Heightgraph.js b/js/plugin/Heightgraph.js
index d50a098..81d0813 100644
--- a/js/plugin/Heightgraph.js
+++ b/js/plugin/Heightgraph.js
@@ -56,6 +56,10 @@ BR.Heightgraph = function(map, layersControl, routing, pois) {
                         color: '#AD0F0C'
                     }
                 }
+            },
+            // extra options
+            shortcut: {
+                toggle: 69 // char code for 'e'
             }
         },
 
@@ -80,6 +84,9 @@ BR.Heightgraph = function(map, layersControl, routing, pois) {
             this._mouseMoveHandlerBound = this.mapMousemoveHandler.bind(this);
             this._mouseoutHandlerBound = this._mouseoutHandler.bind(this);
 
+            L.DomEvent.addListener(document, 'keydown', this._keydownListener, this);
+            this.initCollapse(map);
+
             var self = this;
             var container = $('#elevation-chart');
             $(window).resize(function() {
@@ -107,6 +114,41 @@ BR.Heightgraph = function(map, layersControl, routing, pois) {
             this.update();
         },
 
+        initCollapse: function(map) {
+            var self = this;
+            var onHide = function() {
+                $('#elevation-btn').removeClass('active');
+                // we must fetch tiles that are located behind elevation-chart
+                map._onResize();
+
+                if (this.id && BR.Util.localStorageAvailable() && !self.shouldRestoreChart) {
+                    localStorage.removeItem(this.id);
+                }
+            };
+            var onShow = function() {
+                $('#elevation-btn').addClass('active');
+
+                if (this.id && BR.Util.localStorageAvailable()) {
+                    localStorage[this.id] = 'true';
+                }
+            };
+            // on page load, we want to restore collapse state from previous usage
+            $('#elevation-chart')
+                .on('hidden.bs.collapse', onHide)
+                .on('shown.bs.collapse', onShow)
+                .each(function() {
+                    if (this.id && BR.Util.localStorageAvailable() && localStorage[this.id] === 'true') {
+                        self.shouldRestoreChart = true;
+                    }
+                });
+        },
+
+        _keydownListener: function(e) {
+            if (BR.Util.keyboardShortcutsAllowed(e) && e.keyCode === this.options.shortcut.toggle) {
+                $('#elevation-btn').click();
+            }
+        },
+
         update: function(track, layer) {
             // bring height indicator to front, because of track casing in BR.Routing
             if (this._mouseHeightFocus) {
@@ -123,6 +165,9 @@ BR.Heightgraph = function(map, layersControl, routing, pois) {
                     layer.on('mousemove', this._mouseMoveHandlerBound);
                     layer.on('mouseout', this._mouseoutHandlerBound);
                 }
+
+                if (this.shouldRestoreChart === true) $('#elevation-chart').collapse('show');
+                this.shouldRestoreChart = undefined;
             } else {
                 this._removeMarkedSegmentsOnMap();
                 this._resetDrag();
@@ -135,6 +180,11 @@ BR.Heightgraph = function(map, layersControl, routing, pois) {
                     layer.off('mousemove', this._mouseMoveHandlerBound);
                     layer.off('mouseout', this._mouseoutHandlerBound);
                 }
+
+                if ($('#elevation-chart').hasClass('show')) {
+                    this.shouldRestoreChart = true;
+                }
+                $('#elevation-chart').collapse('hide');
             }
         },
 
@@ -188,8 +238,6 @@ BR.Heightgraph = function(map, layersControl, routing, pois) {
                 previousGradient = currentGradient;
             });
 
-            // TODO when elevation profile is open, the toggle button should be blue, not gray
-
             return [
                 {
                     type: 'FeatureCollection',