ben/brouter-web
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Merge pull request #672 from tbsmark86/bug-clear

Browse source 
Bugfix: Selected edges stick on map after route clear
This commit is contained in:
Norbert Renner 2022-11-25 10:04:30 +01:00 • committed by GitHub
commit f99f3c5a0a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 20 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

8
js/control/TrackAnalysis.js
View file

@ -91,6 +91,14 @@ BR.TrackAnalysis = L.Class.extend({
if (segments.length === 0) { if (segments.length === 0) {
$('#track_statistics').html(''); $('#track_statistics').html('');
if (this.highlightedSegments) {
this.map.removeLayer(this.highlightedSegments);
this.highlightedSegments = null;
}
if (this.highlightedSegment) {
this.map.removeLayer(this.highlightedSegment);
this.highlightedSegment = null;
}
return; return;
} }

12
js/control/TrackMessages.js
View file

@ -77,6 +77,7 @@ BR.TrackMessages = L.Class.extend({
} }
this._destroyTable(); this._destroyTable();
this._destroyEdges();
if (data.length === 0) { if (data.length === 0) {
this.listenMapEvents(layer, false); this.listenMapEvents(layer, false);
@ -146,6 +147,17 @@ BR.TrackMessages = L.Class.extend({
return ele || document.getElementById('datatable'); return ele || document.getElementById('datatable');
}, },
_destroyEdges: function () {
if (this._selectedEdge) {
this._map.removeLayer(this._selectedEdge);
this._selectedEdge = null;
}
if (this._hoveredEdge) {
this._map.removeLayer(this._hoveredEdge);
this._hoveredEdge = null;
}
},
_getColumns: function (headings, data) { _getColumns: function (headings, data) {
var columns = [], var columns = [],
defaultOptions, defaultOptions,