recent notes now use autocomplete instead of select box, closes #36

src/public/javascripts/dialogs/recent_notes.js

@@ -2,12 +2,8 @@
 
 const recentNotes = (function() {
     const dialogEl = $("#recent-notes-dialog");
-    const selectBoxEl = $('#recent-notes-select-box');
+    const searchInputEl = $('#recent-notes-search-input');
-    const jumpToButtonEl = $('#recent-notes-jump-to');
+
-    const addLinkButtonEl = $('#recent-notes-add-link');
-    const addCurrentAsChildEl = $("#recent-notes-add-current-as-child");
-    const addRecentAsChildEl = $("#recent-notes-add-recent-as-child");
-    const noteDetailEl = $('#note-detail');
     // list of recent note paths
     let list = [];
 
@@ -33,93 +29,45 @@ const recentNotes = (function() {
 
         dialogEl.dialog({
             modal: true,
-            width: 800
+            width: 800,
+            height: 400
         });
 
-        selectBoxEl.find('option').remove();
+        searchInputEl.val('');
 
         // remove the current note
         const recNotes = list.filter(note => note !== noteTree.getCurrentNotePath());
 
-        $.each(recNotes, (key, valueNotePath) => {
+        searchInputEl.autocomplete({
-            const noteTitle = noteTree.getNotePathTitle(valueNotePath);
+            source: recNotes.map(notePath => {
+                const noteTitle = noteTree.getNotePathTitle(notePath);
 
-            const option = $("<option></option>")
+                return {
-                .attr("value", valueNotePath)
+                    label: noteTitle,
-                .text(noteTitle);
+                    value: notePath
-
-            // select the first one (most recent one) by default
-            if (key === 0) {
-                option.attr("selected", "selected");
                 }
+            }),
+            minLength: 0,
+            autoFocus: true,
+            select: function (event, ui) {
+                noteTree.activateNode(ui.item.value);
 
-            selectBoxEl.append(option);
+                searchInputEl.autocomplete('destroy');
-        });
-    }
-
-    function getSelectedNotePath() {
-        return selectBoxEl.find("option:selected").val();
-    }
-
-    function getSelectedNoteId() {
-        const notePath = getSelectedNotePath();
-        return treeUtils.getNoteIdFromNotePath(notePath);
-    }
-
-    function setActiveNoteBasedOnRecentNotes() {
-        const notePath = getSelectedNotePath();
-
-        noteTree.activateNode(notePath);
-
                 dialogEl.dialog('close');
+            },
+            focus: function (event, ui) {
+                event.preventDefault();
+            },
+            close: function (event, ui) {
+                searchInputEl.autocomplete('destroy');
+                dialogEl.dialog('close');
+            },
+            create: () => searchInputEl.autocomplete("search", ""),
+            classes: {
+                "ui-autocomplete": "recent-notes-autocomplete"
             }
-
-    function addLinkBasedOnRecentNotes() {
-        const notePath = getSelectedNotePath();
-        const noteId = treeUtils.getNoteIdFromNotePath(notePath);
-
-        const linkTitle = noteTree.getNoteTitle(noteId);
-
-        dialogEl.dialog("close");
-
-        link.addLinkToEditor(linkTitle, '#' + notePath);
-    }
-
-    async function addCurrentAsChild() {
-        await cloning.cloneNoteTo(noteEditor.getCurrentNoteId(), getSelectedNoteId());
-
-        dialogEl.dialog("close");
-    }
-
-    async function addRecentAsChild() {
-        await cloning.cloneNoteTo(getSelectedNoteId(), noteEditor.getCurrentNoteId());
-
-        dialogEl.dialog("close");
-    }
-
-    selectBoxEl.keydown(e => {
-        const key = e.which;
-
-        // to get keycodes use http://keycode.info/
-        if (key === 13)// the enter key code
-        {
-            setActiveNoteBasedOnRecentNotes();
-        }
-        else if (key === 76 /* l */) {
-            addLinkBasedOnRecentNotes();
-        }
-        else if (key === 67 /* c */) {
-            addCurrentAsChild();
-        }
-        else if (key === 82 /* r */) {
-            addRecentAsChild()
-        }
-        else {
-            return; // avoid prevent default
-        }
-
-        e.preventDefault();
         });
+    }
 
     reload();
 
@@ -129,15 +77,6 @@ const recentNotes = (function() {
         e.preventDefault();
     });
 
-    selectBoxEl.dblclick(e => {
-        setActiveNoteBasedOnRecentNotes();
-    });
-
-    jumpToButtonEl.click(setActiveNoteBasedOnRecentNotes);
-    addLinkButtonEl.click(addLinkBasedOnRecentNotes);
-    addCurrentAsChildEl.click(addCurrentAsChild);
-    addRecentAsChildEl.click(addRecentAsChild);
-
     return {
         showDialog,
         addRecentNote,

src/public/stylesheets/style.css

@@ -269,3 +269,7 @@ div.ui-tooltip {
     padding: 2px;
     margin-right: 5px;
 }
+
+.recent-notes-autocomplete {
+    border: 0 !important;
+}

src/views/index.ejs

@@ -151,20 +151,7 @@
     </div>
 
     <div id="recent-notes-dialog" title="Recent notes" style="display: none;">
-      <select id="recent-notes-select-box" size="20" style="width: 100%">
+      <input id="recent-notes-search-input" class="form-control"/>
-      </select>
-
-      <br/><br/>
-
-      <p>
-        <button class="btn btn-sm" id="recent-notes-jump-to">Jump to <kbd>enter</kbd></button>
-        &nbsp;
-        <button class="btn btn-sm" id="recent-notes-add-link">Add link <kbd>l</kbd></button>
-
-        <button class="btn btn-sm" id="recent-notes-add-current-as-child">Add current as child <kbd>c</kbd></button>
-
-        <button class="btn btn-sm" id="recent-notes-add-recent-as-child">Add recent as child <kbd>r</kbd></button>
-      </p>
     </div>
 
     <div id="add-link-dialog" title="Add link" style="display: none;">