Some small refactorings

src/file-renderer/index.js

@@ -10,9 +10,10 @@ String.prototype.endsWith = function (suffix) {
   return this.indexOf(suffix, this.length - suffix.length) !== -1;
 };
 
-function displayFileText(filename, evt, canvas) {
+function renderToCanvas(filename, evt, canvas) {
   const view = jDataView(evt.target.result, 0, evt.size);
   const pattern = new Pattern();
+
   filename = filename.toLowerCase();
   if (filename.endsWith("pes")) {
     pesRead(view, pattern);
@@ -29,15 +30,16 @@ function displayFileText(filename, evt, canvas) {
   pattern.drawShape(canvas);
 }
 
-function handleFileReadAbort(evt) {
+function renderAbortMessage(errorMessageRef) {
-  alert("File read aborted.");
+  errorMessageRef.innerHTML = "Render aborted!";
 }
 
-function handleFileReadError(evt) {
+function renderErrorMessage(errorName, errorMessageRef) {
   let message;
-  switch (evt.target.error.name) {
+  switch (errorName) {
     case "NotFoundError":
-      alert("The file could not be found at the time the read was processed.");
+      message =
+        "The file could not be found at the time the read was processed.";
       break;
     case "SecurityError":
       message = "<p>A file security error occured. This can be due to:</p>";
@@ -46,29 +48,33 @@ function handleFileReadError(evt) {
       message += "<li>Performing too many read calls on file resources.</li>";
       message +=
         "<li>The file has changed on disk since the user selected it.</li></ul>";
-      alert(message);
       break;
     case "NotReadableError":
-      alert(
+      message =
-        "The file cannot be read. This can occur if the file is open in another application."
+        "The file cannot be read. This can occur if the file is open in another application.";
-      );
       break;
     case "EncodingError":
-      alert("The length of the data URL for the file is too long.");
+      message = "The length of the data URL for the file is too long.";
       break;
     default:
-      alert("File error code " + evt.target.error.name);
+      message = "Something wrong happened!";
-  }
+      break;
   }
 
-export default function renderFileToCanvas(fileObject, canvas) {
+  errorMessageRef.innerHTML = message;
+}
+
+export default function renderFileToCanvas(
+  fileObject,
+  canvas,
+  errorMessageRef
+) {
   const reader = new FileReader();
 
-  reader.onloadend = function (x) {
+  reader.onloadend = (evt) => renderToCanvas(fileObject.name, evt, canvas);
-    displayFileText.apply(null, [fileObject.name, x, canvas]);
+  reader.abort = (/** @type {any} */ _) => renderAbortMessage(errorMessageRef);
-  };
+  reader.onerror = (evt) =>
-  reader.abort = handleFileReadAbort;
+    renderErrorMessage(evt.target.error.name, errorMessageRef);
-  reader.onerror = handleFileReadError;
 
   if (fileObject) {
     reader.readAsArrayBuffer(fileObject);

src/lib/CardList.svelte

@@ -3,6 +3,7 @@
 
   export let files = [];
   let canvasRefs = [];
+  let errorMessageRef;
 </script>
 
 {#if files.length !== 0}
@@ -12,8 +13,11 @@
         <canvas bind:this={canvasRefs[i]} class="canvas" />
         <p>{file.name}</p>
       </div>
-      {canvasRefs[i] && renderFileToCanvas(file, canvasRefs[i])}
+      {canvasRefs[i] &&
+        renderFileToCanvas(file, canvasRefs[i], errorMessageRef)}
     {/each}
+    <!-- svelte-ignore a11y-missing-content -->
+    <h1 bind:this={errorMessageRef} />
   </div>
 {/if}