Labels update (#15)

milosmns · Nov 13, 2023 · 43b0fc0 · 43b0fc0
1 parent 7e7a534
commit 43b0fc0
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/gradle.properties b/gradle.properties
@@ -9,6 +9,6 @@ org.gradle.jvmargs=-Xmx4096m
 # Project properties
 config.group = xyz.marinkovic.milos
 config.artifact = codestats
-config.version = 0.9.0
+config.version = 0.10.0
 config.gitHubRepoOwner = milosmns
 config.gitHubRepoName = code-stats
diff --git a/src/commonMain/resources/web/js/script.js b/src/commonMain/resources/web/js/script.js
@@ -417,12 +417,14 @@ document.addEventListener('DOMContentLoaded', function () {
 
     var chartData = prepareBreakdownChartData(metricTimeSeries, breakdownSourceSelector, averageSelector);
     var labelToTimeSeriesList = Object.entries(chartData.yValuesMap);
-    var hideLabelsByDefault = labelToTimeSeriesList.length > 7 && labelToTimeSeriesList.length < 30;
-    var hideLegend = labelToTimeSeriesList.length > 30; // usually useless after 30 labels (hover still works)
+    // usually useless after 50 labels, while hover still works. large datasets are dots so they're ok to display
+    var showLegend = labelToTimeSeriesList.length < 200;
+    var disableLabelsDueToSize = showLegend && (labelToTimeSeriesList.length > 7 && labelToTimeSeriesList.length < 50 || labelToTimeSeriesList.length > 200);
     var hasAverages = chartData.averages.some(item => item !== null);
 
     var datasets = [];
     for (const [sourceKey, timeSeries] of labelToTimeSeriesList) {
+      var disableLabelsDueToLackOfData = timeSeries.every(item => item === null);
       datasets.push({
         label: sourceKey,
         data: timeSeries,
@@ -432,7 +434,7 @@ document.addEventListener('DOMContentLoaded', function () {
         borderColor: [getColorFromKey(sourceKey)],
         borderWidth: 2.5,
         spanGaps: true,
-        hidden: hideLabelsByDefault || timeSeries.every(item => item === null),
+        hidden: disableLabelsDueToSize || disableLabelsDueToLackOfData,
       });
     };
     if (hasAverages) {
@@ -497,7 +499,7 @@ document.addEventListener('DOMContentLoaded', function () {
                 weight: '400'
               }
             },
-            display: !hideLegend
+            display: showLegend
           },
           title: {
             display: true,