Hovercards: Add hideOnTargetClick option & Remove hovercards after calling the detach()

[wellyshen]

Related to #210

Proposed Changes

• Added a hideOnTargetClick option to hide the hovercard when its trigger element is clicked
• detach() now removes attached hovercards
• Update README
• (Extra) Improve TS types
• (Extra) Improve JS docs

Testing Instructions

• Set up this repo by referring to this doc
• Check out this PR
• In your terminal, runcd web/packages/hovercards && npm run build:watch
• Open another terminal, run cd web/packages/hovercards && npm run start

Test the hideOnTargetClick option

• Hover over an avatar, and then click it. The hovercard will be removed
• Change the hideOnTargetClick to false (or remove it). In the left section, the click-to-hide functionality should be canceled

Test the detach method

• Don't hide hovercards by modifying the _hideHovercard function
• In the left section, show some hovercards, and click the "Detach" button. Hovercards will be detached and removed

[github-actions]

Size Change: +1.5 kB (+2.35%)

Total Size: 65.5 kB

Filename	Size	Change
dist/index.esm.js	9.62 kB	+241 B (+2.57%)
dist/index.js	9.76 kB	+238 B (+2.5%)
dist/index.mjs	9.62 kB	+241 B (+2.57%)
dist/index.react.js	11.9 kB	+262 B (+2.25%)
dist/index.react.mjs	11.8 kB	+263 B (+2.27%)
dist/index.react.umd.js	6.89 kB	+136 B (+2.01%)
dist/index.umd.js	5.92 kB	+122 B (+2.11%)

_{compressed-size-action}

[aaronfc]

LGTM and works as described!

A small note: can you review if package-lock.json needs to be updated? When I run npm install I got it updated.

[wellyshen]

LGTM and works as described!

A small note: can you review if package-lock.json needs to be updated? When I run npm install I got it updated.

Thank you for the review. I have updated the file as well.