Fix buggy tooltips.

Author: sebadoom

Diff for: package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "jwt.io",
-  "version": "2.1.1",
+  "version": "2.1.2",
   "repository": {
     "type": "git",
     "url": "https://github.com/jsonwebtoken/jsonwebtoken.github.io"

Diff for: package.json

@@ -11,15 +11,50 @@ import tippy from 'tippy.js';
 
 const timeClaims = ['exp', 'nbf', 'iat', 'auth_time', 'updated_at'];
 
+let instance;
+
 function hideTooltip() {
-  decodedElement._tippy.popper.style.opacity = 0;
+  if(instance) {
+    instance.destroy();
+    instance = null;
+  }
 }
 
-function showTooltip(text) {
-  decodedElement._tippy.popper.querySelector('.tippy-content')
-                .textContent = text;
-  decodedElement._tippy.popperInstance.update();
-  decodedElement._tippy.popper.style.opacity = 1;
+function showTooltip(element, text, placement) {
+  function newTooltip() {
+    element.title = text;
+
+    tippy(element, {
+      placement: placement,
+      arrow: true,
+      performance: true,
+      size: 'large',
+      dynamicTitle: true,
+      arrowTransform: 'scale(0.75)',
+      distance: 10,
+      updateDuration: 100,
+      trigger: 'manual'
+    });  
+
+    return element._tippy;
+  }
+
+  if(instance) {
+    if(instance.reference !== element || 
+       instance.options.placement !== placement) {
+      instance.destroy();
+      instance = newTooltip();
+    } else if(instance.popper.querySelector('.tippy-content').textContent !== 
+              text) {
+      instance.popper.querySelector('.tippy-content').textContent = text;
+    }
+  } else {
+    instance = newTooltip();
+  }
+  
+  if(!instance.state.visible) {
+    instance.show();
+  }
 }
 
 function getTimeText(timeStr) {
@@ -49,12 +84,21 @@ function tooltipHandler(event) {
 
   const claim = matches[1];  
 
+  const element = event.target.tagName === 'SPAN' ? 
+    event.target :
+    event.target.querySelector('span');
+
+  if(!element || element.tagName !== 'SPAN') {
+    hideTooltip();
+    return;
+  }
+
   // If this is a time claim and the mouse cursor is on top of the time,
   // let the time tooltip handle this, do nothing.
   // TODO: merge both tooltip handlers?
   const claimEnd = line.indexOf(':');
   if(result.ch >= claimEnd && timeClaims.includes(claim)) {
-    showTooltip(getTimeText(matches[2]));
+    showTooltip(element, getTimeText(matches[2]), 'right');
     return;
   }
 
@@ -64,25 +108,10 @@ function tooltipHandler(event) {
     return;
   }
 
-  showTooltip(claimText);
+  showTooltip(element, claimText, 'left');
 }
 
 export function setupClaimsTooltip() {  
-  tippy(decodedElement, {
-    placement: 'left',
-    arrow: true,
-    followCursor: true,
-    performance: true,
-    size: 'large',
-    dynamicTitle: true,
-    arrowTransform: 'scale(0.75)',
-    distance: 20,
-    sticky: true,
-    updateDuration: 100
-  });  
-
-  decodedElement._tippy.popper.style.opacity = 0;
-
   payloadElement.addEventListener('mousemove', tooltipHandler);
   headerElement.addEventListener('mousemove', tooltipHandler);
 }

Diff for: src/editor/claims-tooltip.js

@@ -78,7 +78,7 @@ describe('Editor', function() {
       return element._tippy.state.visible;
     }
 
-    expect(await this.page.$eval('#decoded-jwt .output', tippyVisible)).
+    expect(await this.page.$eval('[data-tippy]', tippyVisible)).
       to.be.false;
 
     const iatPos = await this.page.evaluate(() => {
@@ -93,7 +93,7 @@ describe('Editor', function() {
     // Wait for animation
     await this.page.waitFor(2000);
 
-    expect(await this.page.$eval('#decoded-jwt .output', tippyVisible))
+    expect(await this.page.$eval('[data-tippy]', tippyVisible))
       .to.be.true;
   });
 

Diff for: test/functional/editor.js

@@ -11,7 +11,7 @@
     #encoded-jwt.box-content.current
       .input.js-input
     #decoded-jwt.box-content
-      .output(title='placeholder')
+      .output
         .jwt-explained.jwt-header
           p.text-line HEADER:
             span ALGORITHM & TOKEN TYPE