hski-github / spotfire-sankey-diagram

Spotfire mods visualization for Sankey diagram
MIT License
4 stars 4 forks source link

Bar tooltip not showing up #20

Closed f1li closed 3 years ago

f1li commented 3 years ago

While in the previous version of the mods I was seeing also vertical bar tooltips, now I don't see them any longer. Did you decide to remove them? It was a useful feature to me.

hski-github commented 3 years ago

Thanks for reporting. I accidentally broke the vertical bar tooltips with some code change, but in the most recent source code it should be back. Could you please check out latest code and check again?

hski-github commented 3 years ago

Should be fixed since https://github.com/hski-github/spotfire-sankey-diagram/pull/11

f1li commented 3 years ago

OK. Let me download latest code and check. Will update here.

f1li commented 3 years ago

Holger, I downloaded latest code and tried to start the development server. Unfortunately I get the following error. Any clue?

image

hski-github commented 3 years ago

Never done it in this way. README says, first call „npm install“ then „npm run server“

f1li commented 3 years ago

This is exactly what I did. If I move to the previous code dir it's fine (npm run server) and dev server starts correctly. When I jump in the new src dir I get the above issue.

hski-github commented 3 years ago

Please make sure to check out latest version of main branch. And maybe remove node_modules, then call „npm install“ then „npm run server“. This should work and I just checked, that it is working fine for me locally.

hski-github commented 3 years ago

I just check with the latest version of the code. Bar tool tip is shown. Hence closing this issue. Please feel free to reopen if it doesn't work for you with latest code.

f1li commented 3 years ago

I can confirm now this is fine. Thanks a lot. Final question on this: can you help me to change row tooltip count number format? Now I see real with 2 decimal digits but actually those are all integer...so I would like to see only 0 decimal digit. For bar tooltip instead this is already OK.

hski-github commented 3 years ago

See Open Topics in the README