Re-add new tooltip component

xhluca commented 2 years ago

This is an updated attempt for: https://github.com/plotly/dash-core-components/pull/940

Note #941 was also added to this PR in https://github.com/plotly/dash-core-components/pull/982/commits/6a18fd9845e510b669aae06c87f0bacda3bbed49

[x] Image example that uses URLs
[x] Image example that reads images from file system and base64 encodes them
[x] Clientside callback example with URLs
[x] Example with loading state and a time.sleep(1) to simulate a long running callback
[x] Serverside callback example with a nicely formatted tooltip made with HTML and inline CSS with a little box shadow, similar to the style of the nytimes:
[x] Clientside callback example of ^

alexcjohnson commented 2 years ago

@xhlulu please drop package-lock.json from this PR, since you aren't updating package.json. If you do change package.json, note that dash core is now using node 16 / npm 7

xhluca commented 2 years ago

@xhlulu please drop package-lock.json from this PR, since you aren't updating package.json. If you do change package.json, note that dash core is now using node 16 / npm 7

Sure. I removed the commit that added package-lock.json and did a force push

xhluca commented 2 years ago

@alexcjohnson am I supposed to commit NAMESPACE changes as well?

xhluca commented 2 years ago

@alexcjohnson I feel this PR is ready for review. A few questions for review:

any idea why the 27 tests are failing? are we still supporting 2.7?
What name should I give it? It's currently called dcc.AbsoluteTooltip but i'm certain there's a better name than this.

alexcjohnson commented 2 years ago

Closing - will remake shortly in the monorepo

plotly / dash-core-components

Re-add new tooltip component #982