From 922e2bb6a369726ba029dc7dc7031c881780e231 Mon Sep 17 00:00:00 2001
From: Omar Rizwan <omar@omar.website>
Date: Sun, 14 Mar 2021 15:53:04 -0700
Subject: [PATCH] extension: also remove spaces; spaces in filenames are
 annoying, imo

---
 extension/background.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/extension/background.js b/extension/background.js
index 2e66a32..ecb16cf 100644
--- a/extension/background.js
+++ b/extension/background.js
@@ -35,7 +35,10 @@ function pathComponent(path, i) {
 const sanitize = (function() {
   // from https://github.com/parshap/node-sanitize-filename/blob/209c39b914c8eb48ee27bcbde64b2c7822fdf3de/index.js
 
-  var illegalRe = /[\/\?<>\\:\*\|"]/g;
+  // I've added ' ' to the list of illegal characters. it's a
+  // decision whether we want to allow spaces in filenames... I think
+  // they're annoying, so I'm sanitizing them out for now.
+  var illegalRe = /[\/\?<>\\:\*\|" ]/g;
   var controlRe = /[\x00-\x1f\x80-\x9f]/g;
   var reservedRe = /^\.+$/;
   var windowsReservedRe = /^(con|prn|aux|nul|com[0-9]|lpt[0-9])(\..*)?$/i;