From 1dd0791c7eaea23c04c321645c5efbc28988d7c8 Mon Sep 17 00:00:00 2001
From: Tim Ledbetter <timledbetter@gmail.com>
Date: Fri, 15 Sep 2023 18:05:09 +0100
Subject: [PATCH] WebServer: Use human-readable sizes on directory listing

A size is no longer displayed for directories; previously, a size of
4096 was always shown.
---
 Userland/Services/WebServer/Client.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Userland/Services/WebServer/Client.cpp b/Userland/Services/WebServer/Client.cpp
index 901b3cd322..bb0741225d 100644
--- a/Userland/Services/WebServer/Client.cpp
+++ b/Userland/Services/WebServer/Client.cpp
@@ -10,6 +10,7 @@
 #include <AK/Debug.h>
 #include <AK/LexicalPath.h>
 #include <AK/MemoryStream.h>
+#include <AK/NumberFormat.h>
 #include <AK/QuickSort.h>
 #include <AK/StringBuilder.h>
 #include <AK/URL.h>
@@ -310,7 +311,7 @@ ErrorOr<void> Client::handle_directory_listing(String const& requested_path, Str
         TRY(builder.try_append(escape_html_entities(name)));
         TRY(builder.try_append("</a></td><td>&nbsp;</td>"sv));
 
-        TRY(builder.try_appendff("<td>{:10}</td><td>&nbsp;</td>", st.st_size));
+        TRY(builder.try_appendff("<td>{:10}</td><td>&nbsp;</td>", is_directory ? "-" : human_readable_size(st.st_size)));
         TRY(builder.try_append("<td>"sv));
         TRY(builder.try_append(TRY(Core::DateTime::from_timestamp(st.st_mtime).to_string())));
         TRY(builder.try_append("</td>"sv));