Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add tooltips #141

Open
jfix opened this issue Jan 18, 2021 · 4 comments · May be fixed by #258
Open

Add tooltips #141

jfix opened this issue Jan 18, 2021 · 4 comments · May be fixed by #258

Comments

@jfix
Copy link

jfix commented Jan 18, 2021

Me again ;-)

Similar to the Github activity heatmap I'd like to suggest to make it possible to show tooltips with some user-definable contents (based on the data attributes for each day, maybe). I've found tippy and a Svelte integration. I've done some basic testing to see how this works with SVG, it does.

However, I am at a loss as to how to combine your component and this one. As a Svelte newbie, I'd appreciate a pointer so that I could maybe provide a PR, or if this doesn't make sense, an idea on how to combine the two components.

@JohnRSim
Copy link

Don't really need tippy but I'd say update Cell.svelte pass through any extra value. From the heat map and add your hover, tap to it.

I'm going to need this soon so will be sending across a PR example.

@Folyd Folyd linked a pull request Jul 3, 2021 that will close this issue
@Folyd
Copy link

Folyd commented Jul 3, 2021

Hi guys, I have submitted a tooltip prototype implementation without tippyjs, See #258. Not perfect yet, feel free to give it a look.

@Neo-Desktop
Copy link

hey @scottbedard is this something you can merge in?

@tweinreich
Copy link

Since this repo is probably dead, I want to share my fork including SVG-native tooltips. Feel free to use it and please let me know if there is a fork that is still being maintained.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants