Skip to content

Commit 45be9a3

Browse files
committed
Fix point legend icon color bug
1 parent e0c0b4e commit 45be9a3

File tree

1 file changed

+21
-18
lines changed

1 file changed

+21
-18
lines changed

scripts/map.js

Lines changed: 21 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
$(window).on('load', function() {
22
var documentSettings = {};
3-
var markerColors = [];
3+
var group2color = {};
44

55
var polygonSettings = [];
66
var polygonsLegend;
@@ -66,28 +66,30 @@ $(window).on('load', function() {
6666
* column in the spreadsheet.
6767
*/
6868
function determineLayers(points) {
69-
var layerNamesFromSpreadsheet = [];
69+
var groups = [];
7070
var layers = {};
71+
7172
for (var i in points) {
72-
var pointLayerNameFromSpreadsheet = points[i].Group;
73-
if (layerNamesFromSpreadsheet.indexOf(pointLayerNameFromSpreadsheet) === -1) {
74-
markerColors.push(
75-
points[i]['Marker Icon'].indexOf('.') > 0
73+
var group = points[i].Group;
74+
if (group && groups.indexOf(group) === -1) {
75+
// Add group to groups
76+
groups.push(group);
77+
78+
// Add color to the crosswalk
79+
group2color[ group ] = points[i]['Marker Icon'].indexOf('.') > 0
7680
? points[i]['Marker Icon']
77-
: points[i]['Marker Color']
78-
);
79-
layerNamesFromSpreadsheet.push(pointLayerNameFromSpreadsheet);
81+
: points[i]['Marker Color'];
8082
}
8183
}
8284

83-
// if none of the points have named layers or if there was only one name, return no layers
84-
if (layerNamesFromSpreadsheet.length === 0) {
85+
// if none of the points have named layers, return no layers
86+
if (groups.length === 0) {
8587
layers = undefined;
8688
} else {
87-
for (var i in layerNamesFromSpreadsheet) {
88-
var layerNameFromSpreadsheet = layerNamesFromSpreadsheet[i];
89-
layers[layerNameFromSpreadsheet] = L.layerGroup();
90-
layers[layerNameFromSpreadsheet].addTo(map);
89+
for (var i in groups) {
90+
var name = groups[i];
91+
layers[name] = L.layerGroup();
92+
layers[name].addTo(map);
9193
}
9294
}
9395
return layers;
@@ -684,10 +686,11 @@ $(window).on('load', function() {
684686

685687
// Append icons to categories in markers legend
686688
$('#points-legend label span').each(function(i) {
687-
var legendIcon = (markerColors[i].indexOf('.') > 0)
688-
? '<img src="' + markerColors[i] + '" class="markers-legend-icon">'
689+
var g = $(this).text().trim();
690+
var legendIcon = (group2color[ g ].indexOf('.') > 0)
691+
? '<img src="' + group2color[ g ] + '" class="markers-legend-icon">'
689692
: '&nbsp;<i class="fas fa-map-marker" style="color: '
690-
+ markerColors[i]
693+
+ group2color[ g ]
691694
+ '"></i>';
692695
$(this).prepend(legendIcon);
693696
});

0 commit comments

Comments
 (0)