feat: delay hiding the overlay

[chrismwendt]

Resolves #48

[codecov]

Codecov Report

Merging #418 (9b6507f) into master (338ed2b) will increase coverage by 0.12%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #418      +/-   ##
==========================================
+ Coverage   85.91%   86.03%   +0.12%     
==========================================
  Files          14       14              
  Lines         653      659       +6     
  Branches      175      177       +2     
==========================================
+ Hits          561      567       +6     
  Misses         92       92              

Impacted Files	Coverage Δ
src/hoverifier.ts	86.00% <100.00%> (+0.35%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 338ed2b...9b6507f. Read the comment docs.

[olafurpg]

What is the best way to try this out with a local Sourcegraph instance? I'd love to give it a spin.

delay hiding the overlay

There's been recent discussions about using the term "popover" instead of hover/overlay. Not sure if we want to change anything in this repo yet but just wanted to bring it up since the PR title uses the word "overlay". cc/ @jjinnii

[chrismwendt]

To try this locally:

• Check out this PR, then yarn link
• yarn run watch
• cd into sourcegraph/sourcegraph then yarn link "@sourcegraph/codeintellify"
• sg start enterprise-codeintel
• Hover over some tokens

[olafurpg]

Just tried this locally! LGTM 👍 We may want to increase the delay a bit more but lets collect more feedback first.

[jjinnii]

@olafurpg @chrismwendt Thanks for tagging me! Obviously, this PR is merged now but if there's something in our repo that says overlay or for future PR related to the popover, it would be great if we could label "code intel popover" for clarity!