Merge pull request #27 from allen505/fix-4-editable-title

Fixing editable Title for Firefox

manifest.json

@@ -1,7 +1,7 @@
 {
 	"manifest_version": 2,
 	"name": "Save for later",
-	"version": "1.2",
+	"version": "1.2.1",
 
 	"description": "Save tabs in window to be used later",
 

popup/main.css

@@ -71,6 +71,7 @@ body {
   text-align: left;
   float: left;
   flex-grow: 1;
+  white-space: pre-wrap;
 }
 
 .iconBtn {

popup/warehouse.js

@@ -82,6 +82,10 @@ function createList(storeObj) {
           event.preventDefault();
           handleTitleUpdate(obj, saveTitle)
         }
+        if (event.key === ' ') {
+          event.preventDefault();
+          insertTextAtCaret(' ');
+        }
       })
 
       saveTitle.addEventListener('focusout', () => {
@@ -178,7 +182,7 @@ function validateAndUpdateTitle(obj, newTitleText) {
 }
 
 function santizeTitleName(titleText) {
-  const maxTitleLength = 29
+  const maxTitleLength = 40
   if (titleText.length >= maxTitleLength) {
     return titleText.substring(0, maxTitleLength);
   }
@@ -230,6 +234,18 @@ function validateAndOpenSave(obj) {
     });
 }
 
+// Reference: https://stackoverflow.com/a/75505941/4688365
+function insertTextAtCaret(text) {
+  var sel, range;
+  sel = window.getSelection();
+  if (sel && sel.getRangeAt && sel.rangeCount) {
+    range = sel.getRangeAt(0);
+    range.deleteContents();
+    range.insertNode(document.createTextNode(text));
+    range.collapse();
+  }
+}
+
 async function storeIt(tabs) {
   // storeIt takes only the required data from what is provided by the browser and
   // stores it into the storage.local