From b12992a4f72cd9e0f43dc64a80f6452fda01def2 Mon Sep 17 00:00:00 2001 From: Nisarg Shah Date: Mon, 10 Feb 2020 19:22:37 +0530 Subject: [PATCH] Fixed outputTimeTable and outputTimeTableTotals markup The `
` element is not closed correctly right now. This causes the browser to do a bit of error-correction. The error-correction process itself is a bit buggy, and in this case it creates a hierarchy of tables instead of putting tables side-by-side. That causes performance issues when large amount of data is loaded. This change should fix: https://github.com/Jorriss/StatisticsParser/issues/34 --- src/assets/js/statsioparser.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/assets/js/statsioparser.js b/src/assets/js/statsioparser.js index f5d924c..06c3fea 100644 --- a/src/assets/js/statsioparser.js +++ b/src/assets/js/statsioparser.js @@ -353,7 +353,7 @@ function outputTimeTable(timeValues, langTitle, langDuration, elapsedLabel, cpuL //result += '' + numeral(timeValues.elapsed).format('0,0') + ''; result += '' + formatms(timeValues.cpu) + ''; result += '' + formatms(timeValues.elapsed) + ''; - result += '
'; + result += '
'; return result; } @@ -390,7 +390,7 @@ function outputTimeTableTotals(executionValues, compileValues, langCompileTitle, //result += '' + numeral(elapsedTotal).format('0,0') + ''; result += '' + formatms(cpuTotal) + ''; result += '' + formatms(elapsedTotal) + ''; - result += '
'; + result += '
'; return result; }