Merge pull request #180 from Stardown-app/feature-local-notepad

Add a setting for whether to store the notepad's content locally

Author: wheelercj

src/getSetting.js

@@ -19,6 +19,8 @@ import { browser } from './browserSpecific.js';
 const defaultSettings = {
     markupLanguage: 'markdown',
     createTextFragment: true,
+    notepadAppendOrInsert: 'append',
+    notepadStorageLocation: 'sync',
     extractMainContent: true,
     omitNav: true,
     omitFooter: true,
@@ -27,7 +29,6 @@ const defaultSettings = {
     notifyOnSuccess: false,
     selectionFormat: 'Copy selection',
     copyTabsWindows: 'current',
-    notepadAppendOrInsert: 'append',
     notepadContent: '',
     lastUpboardVersion: null,
 

src/settings.html

@@ -131,13 +131,6 @@ <h4>General</h4>
       </select>
       <br />
       <br />
-      <label for="notepadAppendOrInsert">Copying into Stardown's notepad</label>
-      <select id="notepadAppendOrInsert" name="notepadAppendOrInsert">
-        <option value="append">appends</option>
-        <option value="insert">inserts where the cursor is</option>
-      </select>
-      <br />
-      <br />
       <input id="createTextFragment" type="checkbox" checked />
       <label for="createTextFragment">Include a
         <a href="https://developer.mozilla.org/en-US/docs/Web/URI/Fragment/Text_fragments" target="_blank">
@@ -155,6 +148,22 @@ <h4>General</h4>
       <br />
       <br />
       <hr />
+      <h4>Notepad</h4>
+      <label for="notepadAppendOrInsert">Copying into Stardown's notepad</label>
+      <select id="notepadAppendOrInsert" name="notepadAppendOrInsert">
+        <option value="append">appends</option>
+        <option value="insert">inserts where the cursor is</option>
+      </select>
+      <br />
+      <br />
+      <label for="notepadStorageLocation">Store Stardown's notepad content in</label>
+      <select id="notepadStorageLocation" name="notepadStorageLocation">
+        <option value="sync">sync storage (8,100 character limit, syncs between devices)</option>
+        <option value="local">local storage (10,485,000 character limit, does not sync)</option>
+      </select>
+      <br />
+      <br />
+      <hr />
       <h4>Readability</h4>
       <input id="extractMainContent" type="checkbox" checked />
       <label for="extractMainContent">When copying an entire page, try to extract the main content of the page</label>

src/settings.js

@@ -26,8 +26,9 @@ const form = document.querySelector('form');
 const markupLanguageEl = document.querySelector('#markupLanguage');
 const selectionFormatEl = document.querySelector('#selectionFormat');
 const copyTabsWindowsEl = document.querySelector('#copyTabsWindows');
-const notepadAppendOrInsertEl = document.querySelector('#notepadAppendOrInsert');
 const createTextFragmentEl = document.querySelector('#createTextFragment');
+const notepadAppendOrInsertEl = document.querySelector('#notepadAppendOrInsert');
+const notepadStorageLocationEl = document.querySelector('#notepadStorageLocation');
 const extractMainContentEl = document.querySelector('#extractMainContent');
 const omitNavEl = document.querySelector('#omitNav');
 const omitFooterEl = document.querySelector('#omitFooter');
@@ -50,8 +51,15 @@ const resetButton = document.querySelector('#reset');
 initAutosave('markupLanguage', markupLanguageEl, 'value');
 initAutosave('selectionFormat', selectionFormatEl, 'value');
 initAutosave('copyTabsWindows', copyTabsWindowsEl, 'value');
-initAutosave('notepadAppendOrInsert', notepadAppendOrInsertEl, 'value');
 initAutosave('createTextFragment', createTextFragmentEl, 'checked');
+initAutosave('notepadAppendOrInsert', notepadAppendOrInsertEl, 'value');
+initAutosave('notepadStorageLocation', notepadStorageLocationEl, 'value', () => {
+    browser.runtime.sendMessage({
+        destination: 'sidebar',
+        category: 'notepadStorageLocation',
+        notepadStorageLocation: notepadStorageLocationEl.value,
+    });
+});
 initAutosave('extractMainContent', extractMainContentEl, 'checked');
 initAutosave('omitNav', omitNavEl, 'checked');
 initAutosave('omitFooter', omitFooterEl, 'checked');
@@ -69,7 +77,7 @@ initAutosave('jsonDestination', jsonDestinationEl, 'value', () => {
     browser.runtime.sendMessage({
         destination: 'background',
         category: 'jsonDestination',
-        jsonDestination: jsonDestinationEl.value
+        jsonDestination: jsonDestinationEl.value,
     });
 });
 
@@ -98,8 +106,9 @@ async function loadSettings() {
         markupLanguageEl.value = await getSetting('markupLanguage');
         selectionFormatEl.value = await getSetting('selectionFormat');
         copyTabsWindowsEl.value = await getSetting('copyTabsWindows');
-        notepadAppendOrInsertEl.value = await getSetting('notepadAppendOrInsert');
         createTextFragmentEl.checked = await getSetting('createTextFragment');
+        notepadAppendOrInsertEl.value = await getSetting('notepadAppendOrInsert');
+        notepadStorageLocationEl.value = await getSetting('notepadStorageLocation');
         extractMainContentEl.checked = await getSetting('extractMainContent');
         omitNavEl.checked = await getSetting('omitNav');
         omitFooterEl.checked = await getSetting('omitFooter');

src/sidebar.html

@@ -54,6 +54,9 @@
         <textarea id="notepad"></textarea>
       </div>
       <p id="char-count"></p>
+      <button id="sync-limit-button" type="button" style="visibility: hidden">
+        Where does syncing stop?
+      </button>
     </main>
     <script src="./sidebar.js"></script>
   </body>