From 696740844b58cc24cd983a1b87acc9f7a6acaad7 Mon Sep 17 00:00:00 2001 From: Phil Hughes Date: Wed, 3 Aug 2016 11:30:36 +0100 Subject: [PATCH] Made logic simpler by moving away from underscorejs --- app/assets/javascripts/labels_select.js | 20 ++++++++------------ 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/app/assets/javascripts/labels_select.js b/app/assets/javascripts/labels_select.js index c26d902bac1..5eb9def1982 100644 --- a/app/assets/javascripts/labels_select.js +++ b/app/assets/javascripts/labels_select.js @@ -53,7 +53,7 @@ dataType: 'JSON', data: data }).done(function(data) { - var labelCount, template, labelTooltipTitle; + var labelCount, template, labelTooltipTitle, labelTitles; $loading.fadeOut(); $dropdown.trigger('loaded.gl.dropdown'); $selectbox.hide(); @@ -69,20 +69,16 @@ $sidebarCollapsedValue.text(labelCount); if (data.labels.length) { - labelTooltipTitle = _.chain(data.labels) - .map(function (label, i) { - if (i < 5) { - return label.title; - } - }) - .compact() - .values(); + labelTitles = data.labels.map(function(label) { + return label.title; + }); - if (data.labels.length > 5) { - labelTooltipTitle.push('and ' + (data.labels.length - 5) + ' more'); + if (labelTitles.length > 5) { + labelTitles = labelTitles.slice(0, 5); + labelTitles.push('and ' + (data.labels.length - 5) + ' more'); } - labelTooltipTitle = labelTooltipTitle.join(', '); + labelTooltipTitle = labelTitles.join(', '); } else { labelTooltipTitle = ''; $sidebarLabelTooltip.tooltip('destroy');