@abi-software/flatmapvuer 1.6.1 → 1.6.2-beta.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abi-software/flatmapvuer",
-  "version": "1.6.1",
+  "version": "1.6.2-beta.1",
   "license": "Apache-2.0",
   "files": [
     "dist/*",
@@ -43,8 +43,8 @@
     "./src/*": "./src/*"
   },
   "dependencies": {
-    "@abi-software/flatmap-viewer": "^3.2.6",
-    "@abi-software/map-utilities": "^1.2.1",
+    "@abi-software/flatmap-viewer": "^3.2.8",
+    "@abi-software/map-utilities": "^1.2.2-beta.1",
     "@abi-software/sparc-annotation": "0.3.2",
     "@abi-software/svg-sprite": "^1.0.1",
     "@element-plus/icons-vue": "^2.3.1",

package/src/components/FlatmapVuer.vue

@@ -152,7 +152,6 @@ Please use `const` to assign meaningful names to them...
           class: '.maplibregl-canvas',
         }"
         :toolbarOptions="toolbarOptions"
-        :drawnType="drawnType"
         :activeDrawTool="activeDrawTool"
         :activeDrawMode="activeDrawMode"
         :newlyDrawnEntry="drawnCreatedEvent"
@@ -260,7 +259,7 @@ Please use `const` to assign meaningful names to them...
           <div
             class="pathway-location"
             :class="{ open: drawerOpen, close: !drawerOpen }"
-            v-show="!disableUI"
+            v-show="!disableUI && requiresDrawer"
           >
             <div
               class="pathway-container"
@@ -270,27 +269,27 @@ Please use `const` to assign meaningful names to them...
             >
               <svg-legends v-if="!isFC" class="svg-legends-container" />
               <template v-if="showStarInLegend">
-              <el-popover
-                content="Location of the featured dataset"
-                placement="right"
-                :teleported="true"
-                trigger="manual"
-                width="max-content"
-                :offset="-10"
-                popper-class="flatmap-popper flatmap-teleport-popper"
-                :visible="hoverVisibilities[9].value"
-                ref="featuredMarkerPopover"
-              >
-                <template #reference>
-                  <div
-                    v-popover:featuredMarkerPopover
-                    class="yellow-star-legend"
-                    v-html="yellowstar"
-                    @mouseover="showTooltip(9)"
-                    @mouseout="hideTooltip(9)"
-                  ></div>
-                </template>
-              </el-popover>
+                <el-popover
+                  content="Location of the featured dataset"
+                  placement="right"
+                  :teleported="true"
+                  trigger="manual"
+                  width="max-content"
+                  :offset="-10"
+                  popper-class="flatmap-popper flatmap-teleport-popper"
+                  :visible="hoverVisibilities[9].value"
+                  ref="featuredMarkerPopover"
+                >
+                  <template #reference>
+                    <div
+                      v-popover:featuredMarkerPopover
+                      class="yellow-star-legend"
+                      v-html="yellowstar"
+                      @mouseover="showTooltip(9)"
+                      @mouseout="hideTooltip(9)"
+                    ></div>
+                  </template>
+                </el-popover>
               </template>
               <!-- The line below places the yellowstar svg on the left, and the text "Featured markers on the right" with css so they are both centered in the div -->
               <el-popover
@@ -454,44 +453,22 @@ Please use `const` to assign meaningful names to them...
               </template>
             </div>
             <el-row class="viewing-mode-description">
-              {{ viewingModes[viewingMode] }}
+              {{ modeDescription }}
             </el-row>
           </el-row>
           <template v-if="viewingMode === 'Annotation' && userInformation">
-            <el-row class="backgroundText">Drawn By*</el-row>
-            <el-row class="backgroundControl">
-              <el-select
-                :teleported="false"
-                v-model="drawnType"
-                placeholder="Select"
-                class="select-box"
-                popper-class="flatmap_dropdown"
-                @change="setDrawnType"
-              >
-                <el-option
-                  v-for="item in drawnTypes"
-                  :key="item"
-                  :label="item"
-                  :value="item"
-                >
-                  <el-row>
-                    <el-col :span="12">{{ item }}</el-col>
-                  </el-row>
-                </el-option>
-              </el-select>
-            </el-row>
-            <el-row class="backgroundText">Annotated By*</el-row>
+            <el-row class="backgroundText">Annotations From</el-row>
             <el-row class="backgroundControl">
               <el-select
                 :teleported="false"
-                v-model="annotatedType"
+                v-model="annotationFrom"
                 placeholder="Select"
                 class="select-box"
                 popper-class="flatmap_dropdown"
-                @change="setAnnotatedType"
+                @change="setAnnotationFrom"
               >
                 <el-option
-                  v-for="item in annotatedTypes"
+                  v-for="item in annotatedSource"
                   :key="item"
                   :label="item"
                   :value="item"
@@ -773,6 +750,7 @@ export default {
       if (this.annotationSidebar) this.$emit("annotation-close")
       this.closeTooltip()
       this.annotationEventCallback({}, { type: 'aborted' })
+      this.initialiseDrawing()
     },
     /**
      * @public
@@ -885,8 +863,7 @@ export default {
           }
           this.doubleClickedFeature = false
         }
-      }
-      if (this.activeDrawMode === 'Delete') {
+      } else if (this.activeDrawMode === 'Delete') {
         this.changeAnnotationDrawMode({
           mode: 'simple_select',
           options: { featureIds: [data.feature.feature.id] }
@@ -975,6 +952,7 @@ export default {
         if (this.annotationEntry.type === 'deleted') {
           if (this.annotationSidebar) this.$emit("annotation-close")
           this.closeTooltip()
+          // Only delete need, keep the annotation tooltip/sidebar open if created/updated
           this.annotationEntry = {}
         }
         this.addAnnotationFeature()
@@ -1015,12 +993,6 @@ export default {
       let drawnFeatures = await this.annotator.drawnFeatures(this.userToken, this.serverURL, annotatedItemIds)
       // The annotator has `resource` and `features` fields
       if ('resource' in drawnFeatures) drawnFeatures = drawnFeatures.features
-      // Use to switch the displayed feature type
-      if (this.drawnType !== 'All tools') {
-        drawnFeatures = drawnFeatures.filter((feature) => {
-          return feature.geometry.type === this.drawnType
-        })
-      }
       return drawnFeatures
     },
     /**
@@ -1030,21 +1002,19 @@ export default {
     addAnnotationFeature: async function () {
       if (this.mapImp) {
         if (!this.featureAnnotationSubmitted) this.clearAnnotationFeature()
-        if (this.drawnType !== 'None') {
-          if (!this.featureAnnotationSubmitted) this.loading = true
-          const userId = this.annotatedType === 'Anyone' ?
-            undefined : this.userInformation.orcid ?
-              this.userInformation.orcid : '0000-0000-0000-0000'
-          const participated = this.annotatedType === 'Anyone' ?
-            undefined : this.annotatedType === 'Me' ?
-              true : false
-          const drawnFeatures = await this.fetchDrawnFeatures(userId, participated)
-          this.existDrawnFeatures = drawnFeatures
-          this.loading = false
-          if (!this.featureAnnotationSubmitted) {
-            for (const feature of drawnFeatures) {
-              this.mapImp.addAnnotationFeature(feature)
-            }
+        if (!this.featureAnnotationSubmitted) this.loading = true
+        const userId = this.annotationFrom === 'Anyone' ?
+          undefined : this.userInformation.orcid ?
+            this.userInformation.orcid : '0000-0000-0000-0000'
+        const participated = this.annotationFrom === 'Anyone' ?
+          undefined : this.annotationFrom === 'Me' ?
+            true : false
+        const drawnFeatures = await this.fetchDrawnFeatures(userId, participated)
+        this.existDrawnFeatures = drawnFeatures
+        this.loading = false
+        if (!this.featureAnnotationSubmitted) {
+          for (const feature of drawnFeatures) {
+            this.mapImp.addAnnotationFeature(feature)
           }
         }
       }
@@ -1062,26 +1032,13 @@ export default {
         this.$el.querySelector('.maplibregl-ctrl-group').style.display = 'none'
       }
     },
-    /**
-     * @public
-     * Function to switch the type of annotation.
-     * @arg {Boolean} `flag`
-     */
-    setDrawnType: function (flag) {
-      this.drawnType = flag
-      if (this.mapImp) {
-        this.manualAbortedOnClose()
-        this.addAnnotationFeature()
-        this.initialiseDrawing()
-      }
-    },
     /**
      * @public
      * Function to switch the type of person who annotated.
      * @arg {Boolean} `flag`
      */
-    setAnnotatedType: function (flag) {
-      this.annotatedType = flag
+    setAnnotationFrom: function (flag) {
+      this.annotationFrom = flag
       if (this.mapImp) {
         this.manualAbortedOnClose()
         this.addAnnotationFeature()
@@ -1589,7 +1546,7 @@ export default {
         this.annotationEntry = {}
       } else if (data.type === 'modeChanged') {
         if (data.feature.mode === 'direct_select') this.doubleClickedFeature = true
-        if (this.annotationSidebar && data.feature.mode === 'simple_select') {
+        if (this.annotationSidebar && data.feature.mode === 'simple_select' && this.activeDrawMode === 'Deleted') {
           this.annotationEventCallback({}, { type: 'aborted' })
         }
       } else if (data.type === 'selectionChanged') {
@@ -1624,6 +1581,7 @@ export default {
         if (data.type === 'created') this.drawnCreatedEvent = payload
         else this.checkAndCreatePopups(payload)
       }
+      if (data.type === 'updated') this.previousEditEvent = data
       if (data.type === 'deleted') this.previousDeletedEvent = data
       else this.previousDeletedEvent = {}
     },
@@ -1711,6 +1669,14 @@ export default {
               !this.activeDrawTool
             ) {
               this.checkAndCreatePopups(payload)
+              if (this.annotationSidebar && this.previousEditEvent.type === 'updated') {
+                this.annotationEntry = {
+                  ...this.previousEditEvent,
+                  resourceId: this.serverURL
+                }
+                this.annotationEventCallback({}, { type: 'aborted' })
+              }
+              this.previousEditEvent = {}
             }
             this.$emit('resource-selected', payload)
           } else {
@@ -1870,12 +1836,8 @@ export default {
           if (data.feature.featureId && data.feature.models) {
             this.displayTooltip(data.feature.models)
           } else if (data.feature.feature) {
-            // in drawing or edit/delete mode is on or has connectivity
-            if (
-              this.activeDrawTool ||
-              this.activeDrawMode ||
-              Object.keys(this.connectionEntry).length > 0
-            ) {
+            // in drawing or edit/delete mode is on or valid drawn
+            if (this.activeDrawTool || this.activeDrawMode || this.isValidDrawnCreated) {
               this.featureAnnotationSubmitted = false
               this.annotationEntry.featureId = data.feature.feature.id
               if (this.activeDrawTool) {
@@ -2042,7 +2004,11 @@ export default {
             isPathwayContainer(parentElement) ||
             isPathwayContainer(nextElementSibling)
           ) {
-            this.drawerOpen = true;
+            if (this.requiresDrawer) {
+              this.drawerOpen = true;
+            } else {
+              this.helpModeActiveIndex += 1;
+            }
           }
         } else {
           // skip the unavailable tooltips
@@ -2174,8 +2140,7 @@ export default {
         !this.disableUI && (
           (
             this.viewingMode === 'Annotation' &&
-            !this.annotationSidebar &&
-            this.userInformation
+            !this.annotationSidebar
           ) ||
           (
             this.viewingMode === 'Exploration' &&
@@ -2189,7 +2154,6 @@ export default {
           this.mapImp.showPopup(featureId, this.$refs.tooltip.$el, options);
           this.popUpCssHacks();
         });
-
       }
     },
     hasTooltipEntry: function () {
@@ -2483,7 +2447,7 @@ export default {
           }
         }
 
-        let promise1 = this.mapManager.loadMap(
+        let promise1 = this.mapManagerRef.loadMap(
           identifier,
           this.$refs.display,
           this.eventCallback(),
@@ -2568,7 +2532,6 @@ export default {
       this.addResizeButtonToMinimap()
       this.loading = false
       this.computePathControlsMaximumHeight()
-      this.drawerOpen = true;
       this.mapResize()
       this.handleMapClick();
       this.setInitMapState();
@@ -2831,6 +2794,14 @@ export default {
       type: Object,
       default: undefined,
     },
+    /**
+     * Flatmap's Map Manager to use as single Map Manager
+     * if the FlatmapVuer is loaded from MultiFlatmapVuer.
+     */
+    mapManager: {
+      type: Object,
+      default: undefined,
+    },
     /**
      * Specify the endpoint of the flatmap server.
      */
@@ -2878,7 +2849,7 @@ export default {
   data: function () {
     return {
       sensor: null,
-      mapManager: undefined,
+      mapManagerRef: undefined,
       flatmapQueries: undefined,
       annotationEntry: {},
       //tooltip display has to be set to false until it is rendered
@@ -2942,12 +2913,10 @@ export default {
       viewingModes: {
         'Exploration': 'Find relevant research and view detail of neural pathways by selecting a pathway to view its connections and data sources',
         'Neuron Connection': 'Discover Neuron connections by selecting a neuron and viewing its associated network connections',
-        'Annotation': 'View internal identifiers of features'
+        'Annotation': ['View feature annotations', 'Add, comment on and view feature annotations']
       },
-      drawnType: 'All tools',
-      drawnTypes: ['All tools', 'Point', 'LineString', 'Polygon', 'None'],
-      annotatedType: 'Anyone',
-      annotatedTypes: ['Anyone', 'Me', 'Others'],
+      annotationFrom: 'Anyone',
+      annotatedSource: ['Anyone', 'Me', 'Others'],
       openMapRef: undefined,
       backgroundIconRef: undefined,
       toolbarOptions: [
@@ -2964,6 +2933,7 @@ export default {
       activeDrawTool: undefined,
       featureAnnotationSubmitted: false,
       drawnCreatedEvent: {},
+      previousEditEvent: {},
       previousDeletedEvent: {},
       connectionEntry: {},
       existDrawnFeatures: [], // Store all exist drawn features
@@ -2998,6 +2968,37 @@ export default {
     isValidDrawnCreated: function () {
       return Object.keys(this.drawnCreatedEvent).length > 0
     },
+    requiresDrawer: function() {
+      if (this.loading) {
+        this.drawerOpen = false
+        return false
+      }
+      if (!this.isFC) {
+        this.drawerOpen = true
+        return true
+      } else {
+        if ((this.systems?.length > 0) ||
+          (this.containsAlert && this.alertOptions) ||
+          (this.pathways?.length > 0) ||
+          (this.taxonConnectivity?.length > 0)
+        ) {
+          this.drawerOpen = true
+          return true
+        }
+      }
+      this.drawerOpen = false
+      return true
+    },
+    modeDescription: function () {
+      let description = this.viewingModes[this.viewingMode]
+      if (this.viewingMode === 'Annotation') {
+        if (this.userInformation) {
+          return description[1]
+        }
+        return description[0]
+      }
+      return description
+    },
   },
   watch: {
     entry: function () {
@@ -3019,7 +3020,7 @@ export default {
     state: {
       handler: function (state, oldVal) {
         if (state !== oldVal) {
-          if (this.mapManager) {
+          if (this.mapManagerRef) {
             this.setState(state)
           } else {
             //this component has not been mounted yet
@@ -3088,11 +3089,22 @@ export default {
       }
     }
   },
+  created: function () {
+    if (this.mapManager) {
+      this.mapManagerRef = this.mapManager;
+    } else {
+      this.mapManagerRef = markRaw(new flatmap.MapManager(this.flatmapAPI));
+      /**
+       * The event emitted after a new mapManager is loaded.
+       * This mapManager can be used to create new flatmaps.
+       */
+      this.$emit('mapmanager-loaded', this.mapManagerRef);
+    }
+  },
   mounted: function () {
     this.openMapRef = shallowRef(this.$refs.openMapRef)
     this.backgroundIconRef = shallowRef(this.$refs.backgroundIconRef)
     this.tooltipWait.length = this.hoverVisibilities.length
-    this.mapManager = markRaw(new flatmap.MapManager(this.flatmapAPI))
     this.flatmapQueries = markRaw(new FlatmapQueries())
     this.flatmapQueries.initialise(this.flatmapAPI)
     if (this.state) {
@@ -3189,6 +3201,7 @@ export default {
 
 .svg-legends-container {
   width: 70%;
+  min-width:183px;
   height: auto;
   position: relative;
   max-height: 140px;

package/src/components/MultiFlatmapVuer.vue

@@ -83,6 +83,7 @@
       :displayMinimap="displayMinimap"
       :showStarInLegend="showStarInLegend"
       style="height: 100%"
+      :mapManager="mapManagerRef"
       :flatmapAPI="flatmapAPI"
       :sparcAPI="sparcAPI"
     />
@@ -124,6 +125,9 @@ export default {
     Popover,
     FlatmapVuer,
   },
+  created: function () {
+    this.loadMapManager();
+  },
   mounted: function () {
     this.initialise()
     EventBus.on('onActionClick', (action) => {
@@ -219,6 +223,23 @@ export default {
         }
       })
     },
+    /**
+     * Function to load `mapManager` to create flatmap.
+     */
+    loadMapManager: function () {
+      if (!this.mapManagerRef) {
+        if (this.mapManager) {
+          this.mapManagerRef = this.mapManager;
+        } else {
+          this.mapManagerRef = markRaw(new flatmap.MapManager(this.flatmapAPI));
+          /**
+           * The event emitted after a new mapManager is loaded.
+           * This mapManager can be used to create new flatmaps.
+           */
+          this.$emit('mapmanager-loaded', this.mapManagerRef);
+        }
+      }
+    },
     /**
      * @public
      * Function to emit ``resource-selected`` event with provided ``resource``.
@@ -410,12 +431,11 @@ export default {
             //uuid is in the state but should be checked if it is the latest map
             //for that taxon
             return new Promise(() => {
-              const mapManager = new flatmap.MapManager(this.flatmapAPI)
               //mapManager.findMap_ is an async function so we need to wrap this with a promise
               const identifier = { taxon: mapState.entry }
               if (mapState.biologicalSex)
                 identifier['biologicalSex'] = mapState.biologicalSex
-              mapManager
+              this.mapManagerRef
                 .findMap_(identifier)
                 .then((map) => {
                   if (map.uuid !== mapState.uuid) {
@@ -461,6 +481,9 @@ export default {
      */
     setState: function (state) {
       if (state) {
+        // Update undefined mapManagerRef for setState happens before created event
+        this.loadMapManager();
+
         //Update state if required
         this.updateState(state).then((currentState) => {
           this.initialise().then(() => {
@@ -696,6 +719,14 @@ export default {
       type: Object,
       default: undefined,
     },
+    /**
+     * Flatmap's Map Manager to use as single Map Manager
+     * when the value is provided.
+     */
+    mapManager: {
+      type: Object,
+      default: undefined,
+    },
     /**
      * Specify the endpoint of the flatmap server.
      */
@@ -739,6 +770,7 @@ export default {
       requireInitialisation: true,
       resolveList: markRaw([]),
       initialised: false,
+      mapManagerRef: undefined,
     }
   },
   watch: {