Make filenames truncate at 200 characters (#768)

Fixes #766
mozilla · Mar 16, 2018 · 5786932 · 5786932
1 parent ebdd6a9
commit 5786932
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/sidebar/app/utils/utils.js b/src/sidebar/app/utils/utils.js
@@ -31,7 +31,7 @@ function getFirstNonEmptyElement(parentElement) {
 /**
  * Formats the filename for the "Export as HTML..." menu option.
  * Whitespace and illegal filename characters are removed, and
- * the length is shortened if longer than 250 characters.
+ * the length is shortened if longer than 200 characters.
  * @param {string} filename
  * @returns {string}
  */
@@ -41,8 +41,8 @@ function formatFilename(filename) {
   formattedFilename = formattedFilename.trim();
   // remove illegal filename characters
   formattedFilename = formattedFilename.replace(/[~#%{}[\]:\\<>/!@&?"*.+|\n\r\t]/g, '');
-  if (formattedFilename.length > 250) { // 255 bytes (filesystem max) - 5 for ".html" extension
-    formattedFilename = formattedFilename.substring(0, 250);
+  if (formattedFilename.length > 200) { // 200 bytes (close to filesystem max) - 5 for ".html" extension
+    formattedFilename = formattedFilename.substring(0, 200);
   }
   return `${formattedFilename}.html`;
 }