Fix the webserver's HTML correctness

The doctype tag must be provided, otherwise e.g. Firefox logs a warning about running the page in quirks mode in the web console. The `target` attribute being set to `pdf` doesn't do anything and according to MDN is even an unsupported value for the attribute.
timvandermeij · Feb 11, 2024 · c7a98a2 · c7a98a2
1 parent 7e2ff41
commit c7a98a2
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/test/webserver.mjs b/test/webserver.mjs
@@ -211,7 +211,7 @@ WebServer.prototype = {
           return;
         }
         res.write(
-          '<html><head><meta charset="utf-8"></head><body>' +
+          '<!DOCTYPE html><html><head><meta charset="utf-8"></head><body>' +
             "<h1>Index of " +
             pathPart +
             "</h1>\n"
@@ -239,7 +239,6 @@ WebServer.prototype = {
             } else if (path.extname(file).toLowerCase() === ".pdf") {
               href = "/web/viewer.html?file=" + encodeURIComponent(item);
               label = file;
-              extraAttributes = ' target="pdf"';
             } else {
               href = encodeURI(item);
               label = file;