From 209d8cdaebb2adf06b1660526b4f5ede67d24345 Mon Sep 17 00:00:00 2001 From: Ward Peeters Date: Mon, 30 Jan 2017 06:58:30 +0100 Subject: [PATCH] Added toLocalString for better number formatting --- .../audits/dobetterweb/uses-request-compression.js | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/lighthouse-core/audits/dobetterweb/uses-request-compression.js b/lighthouse-core/audits/dobetterweb/uses-request-compression.js index b73dbc16ae4b..8d39bb1bbef7 100644 --- a/lighthouse-core/audits/dobetterweb/uses-request-compression.js +++ b/lighthouse-core/audits/dobetterweb/uses-request-compression.js @@ -75,11 +75,12 @@ class CompressesResponses extends Audit { totalWastedBytes += gzipSavings; const url = URL.getDisplayName(record.url); - const totalKb = Math.round(originalSize / KB_IN_BYTES); + const totalKb = originalSize / KB_IN_BYTES; + const gzipSavingsKb = gzipSavings / KB_IN_BYTES; results.push({ url, - total: `${totalKb} KB`, - gzipSavings: `${Math.round(100 * gzipSize / originalSize)}%`, + total: `${totalKb.toLocaleString()} KB`, + gzipSavings: `${gzipSavingsKb.toLocaleString()} KB`, }); return results; @@ -105,7 +106,7 @@ class CompressesResponses extends Audit { tableHeadings: { url: 'URL', total: 'Original (KB)', - gzipSavings: 'GZIP Savings (%)', + gzipSavings: 'GZIP Savings (KB)', } } }