hoisting improvements

src/public/app/entities/note_short.js

@@ -546,6 +546,11 @@ class NoteShort {
         const labels = this.getLabels('cssClass');
         return labels.map(l => l.value).join(' ');
     }
+
+    getHoistedCssClass() {
+        const labels = this.getLabels('hoistedCssClass');
+        return labels.map(l => l.value).join(' ');
+    }
 }
 
 export default NoteShort;

src/public/app/services/link.js

@@ -78,7 +78,7 @@ function goToLink(e) {
 
     if (notePath) {
         if ((e.which === 1 && e.ctrlKey) || e.which === 2) {
-            appContext.tabManager.openTabWithNote(notePath);
+            appContext.tabManager.openTabWithNoteWithHoisting(notePath);
         }
         else if (e.which === 1) {
             const activeTabContext = appContext.tabManager.getActiveTabContext();
@@ -127,7 +127,7 @@ function linkContextMenu(e) {
         ],
         selectMenuItemHandler: ({command}) => {
             if (command === 'openNoteInNewTab') {
-                appContext.tabManager.openTabWithNote(notePath);
+                appContext.tabManager.openTabWithNoteWithHoisting(notePath);
             }
             else if (command === 'openNoteInNewWindow') {
                 appContext.triggerCommand('openInWindow', {notePath});

src/public/app/services/tab_manager.js

@@ -193,6 +193,24 @@ export default class TabManager extends Component {
         return tabContext;
     }
 
+    /**
+     * If the requested notePath is within current note hoisting scope then keep the note hoisting also for the new tab.
+     */
+    async openTabWithNoteWithHoisting(notePath) {
+        const tabContext = this.getActiveTabContext();
+        let hoistedNoteId = 'root';
+
+        if (tabContext) {
+            const resolvedNotePath = await treeService.resolveNotePath(notePath);
+
+            if (resolvedNotePath.includes(tabContext.hoistedNoteId)) {
+                hoistedNoteId = tabContext.hoistedNoteId;
+            }
+        }
+
+        return this.openTabWithNote(notePath, false, null, hoistedNoteId);
+    }
+
     async openTabWithNote(notePath, activate, tabId, hoistedNoteId) {
         const tabContext = await this.openEmptyTab(tabId, hoistedNoteId);
 

src/public/app/services/tree_context_menu.js

@@ -109,7 +109,7 @@ class TreeContextMenu {
         const notePath = treeService.getNotePath(this.node);
 
         if (command === 'openInTab') {
-            appContext.tabManager.openTabWithNote(notePath);
+            appContext.tabManager.openTabWithNoteWithHoisting(notePath);
         }
         else if (command === "insertNoteAfter") {
             const parentNoteId = this.node.data.parentNoteId;

src/public/app/widgets/note_tree.js

@@ -181,7 +181,7 @@ export default class NoteTreeWidget extends TabAwareWidget {
                 const notePath = treeService.getNotePath(node);
 
                 if (notePath) {
-                    appContext.tabManager.openTabWithNote(notePath);
+                    appContext.tabManager.openTabWithNoteWithHoisting(notePath);
                 }
 
                 e.stopPropagation();
@@ -311,7 +311,7 @@ export default class NoteTreeWidget extends TabAwareWidget {
                     }
                     else if (event.ctrlKey) {
                         const notePath = treeService.getNotePath(node);
-                        appContext.tabManager.openTabWithNote(notePath);
+                        appContext.tabManager.openTabWithNoteWithHoisting(notePath);
                     }
                     else if (data.node.isActive()) {
                         // this is important for single column mobile view, otherwise it's not possible to see again previously displayed note

src/public/app/widgets/owned_attribute_list.js

@@ -32,9 +32,9 @@ export default class OwnedAttributeListWidget extends TabAwareWidget {
     }
 
     renderTitle(note) {
-        const ownedNotes = note.getAttributes().filter(attr => attr.noteId === this.noteId)
+        const ownedAttrs = note.getAttributes().filter(attr => attr.noteId === this.noteId && !attr.isAutoLink)
 
-        this.$title.text(`Owned attrs (${ownedNotes.length})`);
+        this.$title.text(`Owned attrs (${ownedAttrs.length})`);
 
         return {
             show: true,

src/public/app/widgets/type_widgets/relation_map.js

@@ -190,7 +190,7 @@ export default class RelationMapTypeWidget extends TypeWidget {
         const noteId = this.idToNoteId($noteBox.prop("id"));
 
         if (command === "openInNewTab") {
-            appContext.tabManager.openTabWithNote(noteId);
+            appContext.tabManager.openTabWithNoteWithHoisting(noteId);
         }
         else if (command === "remove") {
             const confirmDialog = await import('../../dialogs/confirm.js');

src/services/attributes.js

@@ -18,6 +18,7 @@ const BUILTIN_ATTRIBUTES = [
     { type: 'label', name: 'hidePromotedAttributes' },
     { type: 'label', name: 'readOnly' },
     { type: 'label', name: 'autoReadOnlyDisabled' },
+    { type: 'label', name: 'hoistedCssClass' },
     { type: 'label', name: 'cssClass' },
     { type: 'label', name: 'iconClass' },
     { type: 'label', name: 'keyboardShortcut' },