fix: formatSize() rounding boundary showed "1024 KB" instead of switching to MB

Any file between 1048064-1048575 bytes (a completely realistic PDF
size) rounded via toFixed(0) to display as "1024 KB" rather than
switching to the MB tier, since the branch check compared raw bytes
against 1024*1024 before rounding was applied. Fixed by checking the
rounded KB value against the threshold instead.

Author: Antikode

app.js

@@ -39,7 +39,8 @@ if ("PerformanceObserver" in window) {
 
 function formatSize(bytes) {
   if (bytes < 1024) return `${bytes} B`;
-  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(0)} KB`;
+  const kb = bytes / 1024;
+  if (Math.round(kb) < 1024) return `${kb.toFixed(0)} KB`;
   return `${(bytes / (1024 * 1024)).toFixed(1)} MB`;
 }
 

bookmarks.js

@@ -44,7 +44,8 @@ if ("PerformanceObserver" in window) {
 
 function formatSize(bytes) {
   if (bytes < 1024) return `${bytes} B`;
-  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(0)} KB`;
+  const kb = bytes / 1024;
+  if (Math.round(kb) < 1024) return `${kb.toFixed(0)} KB`;
   return `${(bytes / (1024 * 1024)).toFixed(1)} MB`;
 }
 

border.js

@@ -41,7 +41,8 @@ if ("PerformanceObserver" in window) {
 
 function formatSize(bytes) {
   if (bytes < 1024) return `${bytes} B`;
-  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(0)} KB`;
+  const kb = bytes / 1024;
+  if (Math.round(kb) < 1024) return `${kb.toFixed(0)} KB`;
   return `${(bytes / (1024 * 1024)).toFixed(1)} MB`;
 }
 

compress.js

@@ -54,7 +54,8 @@ if ("PerformanceObserver" in window) {
 
 function formatSize(bytes) {
   if (bytes < 1024) return `${bytes} B`;
-  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(0)} KB`;
+  const kb = bytes / 1024;
+  if (Math.round(kb) < 1024) return `${kb.toFixed(0)} KB`;
   return `${(bytes / (1024 * 1024)).toFixed(1)} MB`;
 }
 

crop.js

@@ -42,7 +42,8 @@ if ("PerformanceObserver" in window) {
 
 function formatSize(bytes) {
   if (bytes < 1024) return `${bytes} B`;
-  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(0)} KB`;
+  const kb = bytes / 1024;
+  if (Math.round(kb) < 1024) return `${kb.toFixed(0)} KB`;
   return `${(bytes / (1024 * 1024)).toFixed(1)} MB`;
 }
 

delete-pages.js

@@ -39,7 +39,8 @@ if ("PerformanceObserver" in window) {
 
 function formatSize(bytes) {
   if (bytes < 1024) return `${bytes} B`;
-  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(0)} KB`;
+  const kb = bytes / 1024;
+  if (Math.round(kb) < 1024) return `${kb.toFixed(0)} KB`;
   return `${(bytes / (1024 * 1024)).toFixed(1)} MB`;
 }
 

duplicate-pages.js

@@ -39,7 +39,8 @@ if ("PerformanceObserver" in window) {
 
 function formatSize(bytes) {
   if (bytes < 1024) return `${bytes} B`;
-  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(0)} KB`;
+  const kb = bytes / 1024;
+  if (Math.round(kb) < 1024) return `${kb.toFixed(0)} KB`;
   return `${(bytes / (1024 * 1024)).toFixed(1)} MB`;
 }
 

extract-images.js

@@ -42,7 +42,8 @@ if ("PerformanceObserver" in window) {
 
 function formatSize(bytes) {
   if (bytes < 1024) return `${bytes} B`;
-  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(0)} KB`;
+  const kb = bytes / 1024;
+  if (Math.round(kb) < 1024) return `${kb.toFixed(0)} KB`;
   return `${(bytes / (1024 * 1024)).toFixed(1)} MB`;
 }
 

fill-form.js

@@ -39,7 +39,8 @@ if ("PerformanceObserver" in window) {
 
 function formatSize(bytes) {
   if (bytes < 1024) return `${bytes} B`;
-  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(0)} KB`;
+  const kb = bytes / 1024;
+  if (Math.round(kb) < 1024) return `${kb.toFixed(0)} KB`;
   return `${(bytes / (1024 * 1024)).toFixed(1)} MB`;
 }
 

flatten.js

@@ -38,7 +38,8 @@ if ("PerformanceObserver" in window) {
 
 function formatSize(bytes) {
   if (bytes < 1024) return `${bytes} B`;
-  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(0)} KB`;
+  const kb = bytes / 1024;
+  if (Math.round(kb) < 1024) return `${kb.toFixed(0)} KB`;
   return `${(bytes / (1024 * 1024)).toFixed(1)} MB`;
 }