Merge pull request #200 from netcreateorg/hotfix-comment-zindex

hotfix-comment-zindex: Keep comments visible on top of comment status…

Author: benloh

app/view/netcreate/NetCreate.css

@@ -1,3 +1,26 @@
+/* Z-Index
+
+      5000  NCDialog
+      3000  nc-savelert
+      2010  NCDialog .dialogWindow
+      2001  NCDialog .screen ==========
+      2000  comment-bar
+      1030  bootstrap .fixed-top
+       100  commentThread
+        15  NCAutoSuggest .matchlist
+        15  NCNode .ncedge
+        10  NCNode .nccomponent
+         1  NCNode .screen ============
+         1  ToolTip .modal, .tooltiptextabove, .tooltiptext
+         1  nc-navbar
+         0  NetCreate
+*/
+
+/* override bootstrap */
+nav.fixed-top {
+  z-index: inherit;
+}
+
 /* Base App: NetCrate.jsx */
 .nc-base {
   display: flex;
@@ -27,7 +50,7 @@
   height: 38px;
   display: flex;
   flex-flow: row-reverse;
-  z-index: 2000;
+  z-index: 1;
 }
 /* Bootstrap override */
 .nav-item {

app/view/netcreate/components/NCComment.css

@@ -232,7 +232,7 @@ svg#comment-icon {
   max-height: calc(80% - 35px);
   font-size: 14px;
   cursor: move;
-  z-index: 100;
+  z-index: 2500;
   /* end floating */
 }
 .commentThread textarea.add {

app/view/netcreate/components/NCNode.jsx

@@ -241,17 +241,17 @@ class NCNode extends UNISYS.Component {
   SetPermissions(data) {
     UDATA.NetCall('SRV_GET_EDIT_STATUS').then(data => {
       // someone else might be editing a template or importing or editing node or edge
-    const { id } = this.state;
-    const nodeIsLocked = data.lockedNodes.includes(id);
-    this.setState(
-      {
-        uIsLockedByDB: nodeIsLocked,
-        uIsLockedByTemplate: data.templateBeingEdited,
+      const { id } = this.state;
+      const nodeIsLocked = data.lockedNodes.includes(id);
+      this.setState(
+        {
+          uIsLockedByDB: nodeIsLocked,
+          uIsLockedByTemplate: data.templateBeingEdited,
           uIsLockedByImport: data.importActive,
           uIsLockedByComment: data.commentBeingEditedByMe
-      },
-      () => this.UpdatePermissions()
-    );
+        },
+        () => this.UpdatePermissions()
+      );
     });
   }
   UpdatePermissions() {