clipper doesn't open new tab if the note is already loaded in existing tab

src/public/javascripts/services/note_detail.js

@@ -76,6 +76,22 @@ function getActiveEditor() {
     }
 }
 
+async function activateOrOpenNote(noteId) {
+    for (const tabContext of tabContexts) {
+        if (tabContext.note && tabContext.note.noteId === noteId) {
+            await tabContext.activate();
+            return;
+        }
+    }
+
+    // if no tab with this note has been found we'll create new tab
+
+    await loadNoteDetail(noteId, {
+        newTab: true,
+        activate: true
+    });
+}
+
 function getTabContexts() {
     return tabContexts;
 }
@@ -544,6 +560,7 @@ export default {
     getTabContexts,
     getActiveTabContext,
     getActiveEditor,
+    activateOrOpenNote,
     clearOpenTabsTask,
     filterTabs,
     openEmptyTab,

src/public/javascripts/services/tree.js

@@ -713,10 +713,7 @@ messagingService.subscribeToMessages(message => {
        reload();
    }
    else if (message.type === 'open-note') {
-       noteDetailService.loadNoteDetail(message.noteId, {
-           newTab: true,
-           activate: true
-       });
+       noteDetailService.activateOrOpenNote(message.noteId);
 
        if (utils.isElectron()) {
            const currentWindow = require("electron").remote.getCurrentWindow();

src/routes/api/clipper.js

@@ -38,7 +38,7 @@ async function addClipping(req) {
 
     const rewrittenContent = await addImagesToNote(images, clippingNote, content);
 
-    await clippingNote.setContent(await clippingNote.getContent() + rewrittenContent);
+    await clippingNote.setContent(await clippingNote.getContent() + '<p>' + rewrittenContent + '</p>');
 
     return {
         noteId: clippingNote.noteId